realm / SwiftLint

A tool to enforce Swift style and conventions.
https://realm.github.io/SwiftLint
MIT License
18.46k stars 2.2k forks source link

Adjust case of major and minor severity for CodeClimate reporting #5523

Closed waitbutY closed 3 months ago

waitbutY commented 3 months ago

Currently Gitlab rejects Swiftlint CodeClimate reports because the MAJOR and MINOR severity strings are all caps. Changing these strings to lower case allows developers to submit Swiftlint CodeClimate json files directly to Gitlab to track code quality issues without manually manipulating the file.

See manual grep fix here: https://forum.gitlab.com/t/code-quality-widget-doesnt-load/90442

SwiftLintBot commented 3 months ago
17 Messages
:book: Linting Aerial with this PR took 1.29s vs 1.25s on main (3% slower)
:book: Linting Alamofire with this PR took 1.81s vs 1.8s on main (0% slower)
:book: Linting Brave with this PR took 10.42s vs 10.43s on main (0% faster)
:book: Linting DuckDuckGo with this PR took 5.55s vs 5.63s on main (1% faster)
:book: Linting Firefox with this PR took 13.04s vs 13.08s on main (0% faster)
:book: Linting Kickstarter with this PR took 12.59s vs 12.62s on main (0% faster)
:book: Linting Moya with this PR took 0.7s vs 0.7s on main (0% slower)
:book: Linting NetNewsWire with this PR took 3.91s vs 3.91s on main (0% slower)
:book: Linting Nimble with this PR took 1.02s vs 1.03s on main (0% faster)
:book: Linting PocketCasts with this PR took 10.2s vs 10.25s on main (0% faster)
:book: Linting Quick with this PR took 0.45s vs 0.45s on main (0% slower)
:book: Linting Realm with this PR took 6.35s vs 6.35s on main (0% slower)
:book: Linting Sourcery with this PR took 3.17s vs 3.17s on main (0% slower)
:book: Linting Swift with this PR took 6.33s vs 6.3s on main (0% slower)
:book: Linting VLC with this PR took 1.67s vs 1.69s on main (1% faster)
:book: Linting Wire with this PR took 23.62s vs 23.75s on main (0% faster)
:book: Linting WordPress with this PR took 15.32s vs 15.4s on main (0% faster)

Generated by :no_entry_sign: Danger

waitbutY commented 3 months ago

@SimplyDanny CHANGELOG entry added, and automated tests fixed!

SimplyDanny commented 3 months ago

As easy as it can be. Thank you!

waitbutY commented 3 months ago

As easy as it can be. Thank you!

loved having the chance to contribute a little!