-
Notifications
You must be signed in to change notification settings - Fork 18
SwiftLint Configuration #148
base: develop
Are you sure you want to change the base?
Conversation
It doesn't look like this automatically lints our Astro code yet (I don't see any Xcode project changes). Can you do that in this PR also? |
This looks good 👍 |
swift/README.md
Outdated
fi | ||
``` | ||
|
||
autocorrect is available by command `swiftlint autocorrect` |
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.
Autocorrect (capitalized)...
|
||
```bash | ||
if which swiftlint >/dev/null; then | ||
swiftlint |
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.
Does this automatically point to the mobify-code-style .swiftlint.yml
file?
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.
No, the swiftlint yaml file lives in the root of your iOS project. To point it to the code style lint file we would have to... make a symlink? We could probably do that as part of a postinstall
npm hook
swift/README.md
Outdated
|
||
autocorrect is available by command `swiftlint autocorrect` | ||
|
||
for more information on this linter you can visit [this](https://github.com/realm/SwiftLint) page |
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.
For (capitalized)...
Capitalize
Adding in SwiftLint configuration to Mobify style guide.
Link to the linter being used: https://github.com/realm/SwiftLint
Changes
TODOs: