-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
SwiftLint integration #90
SwiftLint integration #90
Conversation
ec6702b
to
98f585f
Compare
98f585f
to
5762809
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good! Thanks for this improvement, left few questions
When it comes to auto formatting the code in Xcode for example on save. Is it possible with |
It's possible for sure. There are two ways of doing it, both will require to use binary and some small shell script in Xcode project itself.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for explaining and huge amount of work! Let's for now use only SwiftLint
as it's a big improvement itself.
I've created separate issues to enable each commented out rule:
This PR introduces SwiftLint to the project and fixes majority of lint warnings. SwiftLint is integrated as Xcode Plugin to ensure every contributor is using it.
Resolves #86