realm / realm/SwiftLint

Set minimum required SwiftLint version

Open
#2,592 14 comments 5 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Swift
Stars
19.7k
Forks
2.3k
Avg merge
1d 1h
Merged PRs (30d)
11

Description

Currently, it's possible to set a required SwiftLint version through the swiftlint_version config property. The problem with this is that it makes it impossible to use newer SwiftLint versions with the config, since it expects an exact version. I have many .swiftlint.yml files, and I have manually go through them all every time I upgrade SwiftLint.

I see two possible solutions:

  1. Add a mimimum_swiftlint_version property that enforces that the SwiftLint version is at least the given version.
  2. Make the swiftlint_version property support a semver range, so I could use something like swiftlint_version: '>=0.30.1'.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing how the swiftlint_version property is parsed from .swiftlint.yml files and how the installed SwiftLint version is compared. Determine whether the implementation should add a minimum-version property or support semver ranges, then verify that newer compatible SwiftLint versions are accepted without requiring updates to every configuration file.

Written by the indexing model from the issue text.

Assessment

Tech stack
swift
Domain
tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.