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

[Feature request] Validate version when linting #2074

Closed
kimdv opened this Issue Feb 27, 2018 · 5 comments

Comments

Projects
None yet
2 participants
@kimdv
Copy link
Contributor

kimdv commented Feb 27, 2018

New Issue Checklist

Rule Request

I have been thinking about some minimum version validating when linting, something like fastlane have in their fastfiles. When the version is less than the required version there will be an warning or error.

The reason behind this, can be new rules in newer versions and now all developers are aware of new versions.

@marcelofabri

This comment has been minimized.

Copy link
Collaborator

marcelofabri commented Mar 27, 2018

You can already use swiftlint_version on the configuration file.

For example:

swiftlint_version: 0.30.0
$ swiftlint
Loading configuration from '.swiftlint.yml'
Currently running SwiftLint 0.25.0 but configuration specified version 0.30.0.
@kimdv

This comment has been minimized.

Copy link
Contributor

kimdv commented Mar 27, 2018

Is there somehow we can get an inside Xcode error?

@marcelofabri

This comment has been minimized.

Copy link
Collaborator

marcelofabri commented Mar 28, 2018

Not currently. We could potentially make the script fail when that happens.

@kimdv

This comment has been minimized.

Copy link
Contributor

kimdv commented Mar 29, 2018

Or that. Make a new settings that can enable that so it is opt-in

@kimdv

This comment has been minimized.

Copy link
Contributor

kimdv commented Dec 6, 2018

Closing as PR is merged

@kimdv kimdv closed this Dec 6, 2018

@benweitzman benweitzman referenced this issue Dec 9, 2018

Closed

Build fails with swiftlint 0.29.1 #300

5 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment