search

bitwarden / ios

Bitwarden mobile app for iOS.
https://bitwarden.com
GNU General Public License v3.0
199 stars 25 forks source link

[deps]: Update realm/SwiftLint to v0.56.1 #838

Closed renovate[bot] closed 2 months ago

renovate[bot] commented 2 months ago

Mend Renovate

This PR contains the following updates:

Package Update Change
realm/SwiftLint minor 0.55.1 -> 0.56.1

Release Notes

realm/SwiftLint (realm/SwiftLint) ### [`v0.56.1`](https://togithub.com/realm/SwiftLint/blob/HEAD/CHANGELOG.md#0561-Heat-Pump-Dryer) [Compare Source](https://togithub.com/realm/SwiftLint/compare/0.56.0...0.56.1) ##### Breaking - None. ##### Experimental - None. ##### Enhancements - None. ##### Bug Fixes - Let `contrasted_opening_brace` be an opt-in rule.\ [SimplyDanny](https://togithub.com/SimplyDanny) ### [`v0.56.0`](https://togithub.com/realm/SwiftLint/blob/HEAD/CHANGELOG.md#0560-Heat-Pump-Dryer) [Compare Source](https://togithub.com/realm/SwiftLint/compare/0.55.1...0.56.0) ##### Breaking - The deprecated `--path` and `--in-process-sourcekit` arguments have now been removed completely.\ [Martin Redington](https://togithub.com/mildm8nnered) [SimplyDanny](https://togithub.com/SimplyDanny) [#​5614](https://togithub.com/realm/SwiftLint/issues/5614) - When SwiftLint corrects violations automatically (`swiftlint lint --fix`) it doesn't report the exact location of the fix any longer. The new format is `: Correcting ` without line and column numbers. Reason: Correction positions are likely just incorrect, especially when multiple rules apply their rewrites. Fixing that is not trivial and likely not worth the effort also considering that there haven't been any bug reports about wrong correction positions so far.\ [SimplyDanny](https://togithub.com/SimplyDanny) ##### Experimental - None. ##### Enhancements - Linting got up to 30% faster due to the praisworthy performance improvements done in the [SwiftSyntax](https://togithub.com/swiftlang/swift-syntax) library. - Rewrite the following rules with SwiftSyntax: - `missing_docs` [woxtu](https://togithub.com/woxtu)\ [SimplyDanny](https://togithub.com/SimplyDanny) - Add new `prefer_key_path` rule that triggers when a trailing closure on a standard function call is only hosting a (chained) member access expression since the closure can be replaced with a key path argument. Likewise, it triggers on closure arguments.\ [SimplyDanny](https://togithub.com/SimplyDanny) - Adds `baseline` and `write_baseline` configuration file settings, equivalent to the `--baseline` and `--write-baseline` command line options.\ [Martin Redington](https://togithub.com/mildm8nnered) [#​5552](https://togithub.com/realm/SwiftLint/issues/5552) - Add `no_empty_block` opt-in rule to validate that code blocks are not empty. They should at least contain a comment.\ [Ueeek](https://togithub.com/Ueeek) [#​5615](https://togithub.com/realm/SwiftLint/issues/5615) - Add new `contrasted_opening_brace` rule that enforces opening braces to be on a separate line after the preceding declaration.\ [SimplyDanny](https://togithub.com/SimplyDanny) - Add new `unused_parameter` rule that triggers on function/initializer/subscript parameters that are not used inside of the function/initializer/subscript.\ [SimplyDanny](https://togithub.com/SimplyDanny) [#​2120](https://togithub.com/realm/SwiftLint/issues/2120) - Support `--target` paths being passed to command plugin by Xcode.\ [SimplyDanny](https://togithub.com/SimplyDanny) [#​5603](https://togithub.com/realm/SwiftLint/issues/5603) - Add modified configurations to examples in rule documentation.\ [SimplyDanny](https://togithub.com/SimplyDanny) - Add new option `evaluate_effective_access_control_level` to `missing_docs` rule. Setting it to `true` stops the rule from triggering on declarations inside of types with lower visibility. These declarations effectively have at most the same access level.\ [SimplyDanny](https://togithub.com/SimplyDanny) - Add new `--check-for-updates` command line option for the `lint`, `analyze`, and `version` subcommands to check for new versions of SwiftLint, and an equivalent `check_for_updates` configuration file setting.\ [Martin Redington](https://togithub.com/mildm8nnered) [SimplyDanny](https://togithub.com/SimplyDanny) [Ian Leitch](https://togithub.com/ileitch) [#​5613](https://togithub.com/realm/SwiftLint/issues/5613) - Add new `--only-rule` command line option for the `lint` and `analyze`, subcommands that overrides configuration file rule enablement and disablement, in particular to facilitate running `--fix` for single rules without having to temporarily edit the configuration file. [Martin Redington](https://togithub.com/mildm8nnered) [#​5666](https://togithub.com/realm/SwiftLint/issues/5666) ##### Bug Fixes - Fix a few false positives and negatives by updating the parser to support Swift 6 with all its new language constructs.\ [SimplyDanny](https://togithub.com/SimplyDanny) - Stop triggering `mark` rule on "mark" comments in the middle of another comment.\ [SimplyDanny](https://togithub.com/SimplyDanny) [#​5592](https://togithub.com/realm/SwiftLint/issues/5592) - Don't consider specialized imports with attributes as duplicates in `duplicate_imports` rule.\ [SimplyDanny](https://togithub.com/SimplyDanny) [#​5716](https://togithub.com/realm/SwiftLint/issues/5716) - Use correct types and relative paths in SARIF reporter output. Generally avoid escaping slashes in JSON output as well.\ [SimplyDanny](https://togithub.com/SimplyDanny) [#​5598](https://togithub.com/realm/SwiftLint/issues/5598) [#​5599](https://togithub.com/realm/SwiftLint/issues/5599) - Keep initializers with attributed parameters in `unneeded_synthesized_initializer` rule.\ [SimplyDanny](https://togithub.com/SimplyDanny) [#​5153](https://togithub.com/realm/SwiftLint/issues/5153) - Make `vertical_whitespace_between_cases` rule work for cases ending with a string literal.\ [ilendemli](https://togithub.com/ilendemli) [#​5612](https://togithub.com/realm/SwiftLint/issues/5612) - Ignore access level modifiers restricted to value setting in `extension_access_modifier` rule.\ [SimplyDanny](https://togithub.com/SimplyDanny) [#​5623](https://togithub.com/realm/SwiftLint/issues/5623) - Fix `baseline compare` incorrectly reporting some violations as new, and also now sorts the violations from `baseline compare` deterministically.\ [Martin Redington](https://togithub.com/mildm8nnered) [#​5606](https://togithub.com/realm/SwiftLint/issues/5606) - Fix rewriting for `implicit_return` rule when violations are nested within each other.\ [Martin Redington](https://togithub.com/mildm8nnered) [#​5660](https://togithub.com/realm/SwiftLint/issues/5660) - Fix `opening_brace` correction and make sure that disable commands are taken into account before applying a fix.\ [swiftty](https://togithub.com/swiftty) [SimplyDanny](https://togithub.com/SimplyDanny) [#​5598](https://togithub.com/realm/SwiftLint/issues/5598) - Violations of the `typesafe_array_init` rule will now be correctly reported as such, instead of as violations of the `array_init` rule.\ [Martin Redington](https://togithub.com/mildm8nnered) [#​5709](https://togithub.com/realm/SwiftLint/issues/5709)

Configuration

📅 Schedule: Branch creation - "every 2nd week starting on the 2 week of the year before 4am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

This PR was generated by Mend Renovate. View the repository job log.

github-actions[bot] commented 2 months ago

Logo Checkmarx One – Scan Summary & Details – 31622803-20d0-4e46-b6b9-847d05eb0a61

No New Or Fixed Issues Found

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.52%. Comparing base (98ed38e) to head (73419f4). Report is 13 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #838 +/- ## ======================================= Coverage 88.52% 88.52% ======================================= Files 606 606 Lines 30259 30259 ======================================= Hits 26786 26786 Misses 3473 3473 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.