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

Stringify swiftlint_version #2523

Merged
merged 1 commit into from
Dec 24, 2018
Merged

Stringify swiftlint_version #2523

merged 1 commit into from
Dec 24, 2018

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented Dec 23, 2018

to treat any specified value that doesn't match the version string to fail the lint.

Previously if swiftlint_version: 0.29 was specified in the configuration file, any SwiftLint version successfully ran because technically no string was specified as a yaml value.

Now SwiftLint will log the following and exit:

Currently running SwiftLint 0.29.1 but configuration specified version 0.29.

Fixes #2518.

to treat any specified value that doesn't match the version string
to fail the lint.
@SwiftLintBot
Copy link

12 Messages
📖 Linting Aerial with this PR took 2.13s vs 2.12s on master (0% slower)
📖 Linting Alamofire with this PR took 4.62s vs 4.76s on master (2% faster)
📖 Linting Firefox with this PR took 16.53s vs 16.49s on master (0% slower)
📖 Linting Kickstarter with this PR took 26.56s vs 23.94s on master (10% slower)
📖 Linting Moya with this PR took 2.44s vs 2.21s on master (10% slower)
📖 Linting Nimble with this PR took 2.45s vs 2.0s on master (22% slower)
📖 Linting Quick with this PR took 0.71s vs 0.64s on master (10% slower)
📖 Linting Realm with this PR took 4.59s vs 4.08s on master (12% slower)
📖 Linting SourceKitten with this PR took 1.44s vs 1.3s on master (10% slower)
📖 Linting Sourcery with this PR took 5.87s vs 5.42s on master (8% slower)
📖 Linting Swift with this PR took 38.06s vs 32.22s on master (18% slower)
📖 Linting WordPress with this PR took 27.26s vs 23.35s on master (16% slower)

Generated by 🚫 Danger

@jpsim jpsim merged commit ce8ea70 into master Dec 24, 2018
@jpsim jpsim deleted the stringify-version branch December 24, 2018 00:29
sjavora pushed a commit to sjavora/SwiftLint that referenced this pull request Mar 9, 2019
to treat any specified value that doesn't match the version string
to fail the lint.
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.

2 participants