-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Swift: Major improvements #3022
Swift: Major improvements #3022
Conversation
JS File Size Changes (gzipped)A total of 2 files have changed, with a combined diff of +211 B (+6.4%).
|
It's fair to say yes, but Swift doesn't enforce us doing so. I haven't seen any code examples though in which developers defined classes starting with a lowercase! |
Really impressive PR! Thanks a lot, I'm learning a lot about this library as well now 🙂 Btw, if you like to explore how Prism is behaving with Swift, you can check out my articles here: avanderlee.com |
@AvdLee I think I got everything from #3021 now. Also, in your article about dependency injection, you presented this attribute: |
Dang, that's amazing! Thanks a ton 💪
It's an accessor to a key path reference. See "Key-Path Expression" in this document: https://docs.swift.org/swift-book/ReferenceManual/Expressions.html |
Seems like I missed that part while scanning through the docs. Thank you! |
I made a number of changes to Swift:
/* /* foo */ */
).""" foo """
) and raw strings (#" foo "#
)_
.foo: while ...
)Edit:
@AvdLee
I hope this helps with #3021.
Also, I'm not familiar with Swift and not sure about the naming-based class detection. Can we assume that everything starting with an upper-case letter and containing a lower-case letter is a class name?