Skip to content
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

Added swift formatter, skipping unused check, and commit hooks #21

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Panajev
Copy link

@Panajev Panajev commented Oct 12, 2018

Finding this tool useful @s4cha and wanted to contribute a bit since I am using it in projects now :).

I have added a swiftlint configuration file, a script that runs swiftlint in autocorrect mode as well as swiftformat (configured to complement and not work against swiftlint... added a Dangerfile too for Danger integration that could be used with Travis/CircleCI/etc....). Pretty much what I do on other projects, hopefully it is useful.

Formatting Rules
Link to rules for swiftformat and swiftlint (rules opted in and opted out can be seen in the script and in the swiftlint .yml configuration file):

Danger integration notes (optional)

@Panajev
Copy link
Author

Panajev commented Nov 21, 2018

Any update?

Signed-off-by: Goffredo Marocchi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants