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

Add strict configuration file setting #5226

Conversation

mildm8nnered
Copy link
Collaborator

Resolves #5220

Adds a strict configuration file setting, equivalent to the --strict command line option.

If --lenient is passed on the command line, this will take precendence over the configuration file.

@SwiftLintBot
Copy link

SwiftLintBot commented Sep 18, 2023

17 Messages
📖 Linting Aerial with this PR took 1.18s vs 1.18s on main (0% slower)
📖 Linting Alamofire with this PR took 1.58s vs 1.59s on main (0% faster)
📖 Linting Brave with this PR took 9.0s vs 9.04s on main (0% faster)
📖 Linting DuckDuckGo with this PR took 4.39s vs 4.38s on main (0% slower)
📖 Linting Firefox with this PR took 10.4s vs 10.42s on main (0% faster)
📖 Linting Kickstarter with this PR took 11.54s vs 11.59s on main (0% faster)
📖 Linting Moya with this PR took 0.62s vs 0.62s on main (0% slower)
📖 Linting NetNewsWire with this PR took 3.34s vs 3.36s on main (0% faster)
📖 Linting Nimble with this PR took 0.83s vs 0.83s on main (0% slower)
📖 Linting PocketCasts with this PR took 8.93s vs 8.96s on main (0% faster)
📖 Linting Quick with this PR took 0.4s vs 0.4s on main (0% slower)
📖 Linting Realm with this PR took 12.32s vs 12.34s on main (0% faster)
📖 Linting Sourcery with this PR took 2.83s vs 2.86s on main (1% faster)
📖 Linting Swift with this PR took 5.31s vs 5.35s on main (0% faster)
📖 Linting VLC with this PR took 1.56s vs 1.55s on main (0% slower)
📖 Linting Wire with this PR took 9.22s vs 9.19s on main (0% slower)
📖 Linting WordPress with this PR took 13.35s vs 13.39s on main (0% faster)

Generated by 🚫 Danger

Copy link
Collaborator

@SimplyDanny SimplyDanny left a comment

Choose a reason for hiding this comment

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

Thank you!

@SimplyDanny SimplyDanny merged commit f783b02 into realm:main Sep 18, 2023
5 checks passed
@mildm8nnered mildm8nnered deleted the mildm8nnered-add-strict-option-to-config-file branch September 18, 2023 18:12
KS1019 pushed a commit to KS1019/SwiftLint that referenced this pull request Oct 2, 2023
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.

Error severity for every rule using .swiftlint.yml
3 participants