Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

SwiftLint Configuration #148

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Conversation

sshin-mobify
Copy link

Adding in SwiftLint configuration to Mobify style guide.
Link to the linter being used: https://github.com/realm/SwiftLint

Changes

  • updated CHANGELOG
  • added in .swiftlin.yml that is currently configured to follow Mobify style guide

TODOs:

  • Edit the configuration file with other Swift developers

@jeremywiebe
Copy link
Contributor

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?

@jeremywiebe jeremywiebe reopened this Jan 17, 2017
@kerrmarin
Copy link
Contributor

This looks good 👍

swift/README.md Outdated
fi
```

autocorrect is available by command `swiftlint autocorrect`
Copy link
Contributor

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
Copy link
Contributor

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?

Copy link
Contributor

@kerrmarin kerrmarin Mar 3, 2017

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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For (capitalized)...

Capitalize
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

3 participants