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 swiftlint.yml #1

Merged
merged 1 commit into from Mar 29, 2020
Merged

Conversation

bielikb
Copy link
Collaborator

@bielikb bielikb commented Mar 25, 2020

Swiftlint brings conventions, best practices and uniformity to the codebase

Changes:

  • Added swiftlint.yml
  • swiftlint is now part of the project target's build phase
  • fixed couple of swiftlint warnings

Copy link
Collaborator

@anonymcek anonymcek left a comment

Choose a reason for hiding this comment

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

to je nejaky standard ktory pouzivas? vyzera to v pohodke

@bielikb
Copy link
Collaborator Author

bielikb commented Mar 29, 2020

jj, nie je to ani moc utiahnute za vlasy (severity je warning), takze by to malo byt v poho aj pre teba a dalsich zaujemcov.
Po mergnuti uvidis nove warningy (todocka a pod), ktore si nechal v kode.

Az by tento konfigurak vytvaral friction, mozme ho vzdy pozmenit, davnejsie som ho vytvoril vdaka followovaniu guidelineov z LinkedInu a pod.

@anonymcek anonymcek merged commit a5f60af into CovidWorld:master Mar 29, 2020
bielikb added a commit that referenced this pull request Apr 23, 2020
bielikb added a commit that referenced this pull request May 3, 2020
bielikb added a commit that referenced this pull request May 6, 2020
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