Update dependency realm/SwiftLint to v0.63.2#21
Open
renovate[bot] wants to merge 1 commit intomainfrom
Open
Conversation
1f67eeb to
28c00b2
Compare
28c00b2 to
70772a0
Compare
70772a0 to
982ca61
Compare
982ca61 to
c7e2df9
Compare
c7e2df9 to
07e2ffe
Compare
07e2ffe to
4b3c11a
Compare
4b3c11a to
4d32ef2
Compare
4d32ef2 to
f882480
Compare
f882480 to
dfeb097
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
0.58.2→0.63.2Release Notes
realm/SwiftLint (realm/SwiftLint)
v0.63.2Compare Source
Breaking
Experimental
Enhancements
ignore_regexconfiguration option to thelarge_tuplerule to silenceviolations for tuples inside
Regex<...>types, which commonly have largetuple type parameters for capture groups.
Deco354
#6340
Bug Fixes
if violations are disabled for it in
trailing_closurerule.SimplyDanny
#6451
v0.63.1Compare Source
Breaking
Experimental
Enhancements
--disable-sourcekitflag to thelintcommand to disable SourceKit when needed.The environment variable
SWIFTLINT_DISABLE_SOURCEKITcan still be used as well.SimplyDanny
#6282
Bug Fixes
Retain
asyncinitializers in actors inasync_without_awaitrule.SimplyDanny
#6423
Inform users about files being skipped due to impossible file system representation
instead of crashing.
SimplyDanny
#6419
Ignore
overridefunctions inasync_without_awaitrule.SimplyDanny
#6416
Avoid infinite recursion for deeply nested symbolic links which is usually the case
in
node_modulesdirectories managed bypnpm.SimplyDanny
#6425
Fix false positive in
unneeded_escapingrule when an escaping closure is used ina nested closure preceded by another closure.
SimplyDanny
#6410
Fix non-excluded bool literal in
optional_enum_case_namewhen used inside a tuple.tristan-burnside-anz
v0.63.0Compare Source
Breaking
redundant_self_in_closurerule has been renamed toredundant_self(withredundant_self_in_closureas a deprecated alias) to reflect its now broader scope,while by default still maintains the previous behavior of only checking closures.
To enable checking for all redundant
selfusages, set the newonly_in_closuresoption to
false.SimplyDanny
Experimental
Enhancements
Extend
redundant_self_in_closurerule to detect all redundant uses ofself,not just in closures. Initializers (which commonly prefer an explicit
selfprefix)can be ignored by setting
keep_in_initializerstotrue.SimplyDanny
Add a
separationconfiguration option to thevertical_whitespace_between_casesruleto allow customizing blank line separation between switch cases. The default value is
always(require at least one blank line). Setting it toneverenforces no blanklines between cases.
SimplyDanny
#6326
Rewrite the following rules with SwiftSyntax:
vertical_whitespace_between_casesSimplyDanny
Add new opt-in
unneeded_throws_rethrowsrule that triggers when declarationsmarked
throws/rethrowsnever actually throw or call any throwing code.Tony Ngo
Add new
unneeded_escapingrule that detects closure parameters marked with@escapingthat are never stored or captured escapingly.SimplyDanny
Add
multiline_call_argumentsopt-in rule to enforce consistent multilineformatting for function and method call arguments.
GandaLF2006
Improve performance when exclude patterns resolve to a large set of files.
SimplyDanny
#5018
#5207
#5953
#6084
#6259
Bug Fixes
Fix
line_lengthrule incorrectly ignoring function bodies whenignores_function_declarationsis enabled. The option should only ignorefunction declarations, not their implementation.
SimplyDanny
#6347
Fix false positives in
vertical_whitespace_between_casesrule when cases areinterleaved with compiler directives like
#if/#else/#endif.SimplyDanny
#6332
override_in_extensionno longer triggers inside@objc @​implementationextensions.JaredGrubb
Reinstate handling of access level imports in
sorted_importsrule.Brett-Best
#6374
v0.62.2Compare Source
Breaking
Experimental
Enhancements
Add new
incompatible_concurrency_annotationrule that triggers when a declarationisolated to a global actor,
@Sendableclosure arguments and/or generic sendableconstraints is not annotated with
@preconcurrencyin order to maintain compatibilitywith Swift 5.
mattmassicotte
SimplyDanny
#5987
Add
isolationmodifier group tomodifier_orderrule configuration.This allows configuring the position of
nonisolatedmodifiers.nandhinisubbu
#6164
Add test count to JUnit reporter.
nandhinisubbu
#6161
Adopt mimalloc for static Linux binary
to improve performance.
ainame
#6298
Bug Fixes
Fix false positive with
#Previewmacro inclosure_end_indentationrule.gibachan
Fix correction of
sorted_importsrule when comments (with a distance) are present beforethe first import.
SimplyDanny
#6317
v0.62.1Compare Source
Breaking
Experimental
Enhancements
Bug Fixes
SimplyDanny
v0.62.0Compare Source
Breaking
The structure of
SwiftLintBinary.artifactbundle.zipis now simpler. Internal paths nolonger contain version numbers, especially. So in an Xcode Run Script build phase,
you can refer to the
swiftlintbinary like this:All other consumers of the artifact bundle do not need to change anything. Swift Package
Manager resolves the new paths automatically.
SimplyDanny
SwiftLint now requires a Swift 6 or higher compiler to build. The
Swift Package Manager plugins continue
to work with all versions down to Swift 5.9.
SimplyDanny
sorted_importsrule's behavior changed in that it now treats imports directly adjacentto each other as a group that is sorted together. Imports separated by at least one
non-import statement or empty line(s) are treated as separate groups. Comments are
considered part of the import group they are attached to and do not break it.
SimplyDanny
Experimental
Enhancements
Rewrite the following rules with SwiftSyntax:
modifier_ordersorted_importsSimplyDanny
Add new
prefer_asset_symbolsrule that suggests using asset symbols overstring-based image initialization to avoid typos and enable compile-time
checking. This rule detects
UIImage(named:)andSwiftUI.Image(_:)callswith string literals and suggests using asset symbols instead.
danglingP0inter
#5939
Exclude integer generic parameters from
generic_type_namerule.Include integer generic parameters in the
identifier_namerule for validation.nandhinisubbu
#6213
Add new option
ignores_regex_literalstoline_lengthrule.It allows to ignore regex literals.
nandhinisubbu
#6304
Add
ignores_literalsconfiguration fortrailing_whitespacerule.It allows to ignore trailing whitespace in multiline strings.
nandhinisubbu
#6194
Bug Fixes
Ignore function, initializer and subscript declarations alike when the
ignores_function_declarationsoption is enabled in theline_lengthrule.SimplyDanny
#6241
Individual
custom_rulescan now be specified in theonly_ruleconfigurationsetting and the
--only-rulecommand line option without having to specifycustom_rulesas well. Additionally, violations of custom rules are now reportedin a deterministic order, sorted by the rule's identifier.
Martin Redington
#6029
#6058
Ignore
redundant_discardable_letrule violations in nested SwiftUI scopes as wellwhen
ignore_swiftui_view_bodiesis enabled.SimplyDanny
#3855
#6255
Exclude function types from
async_without_awaitrule analysis. Higher-order functionobjects can be
asyncwithout containing anawaitwhen assigning to them.SimplyDanny
#6253
Exclude
@concurrentfunctions fromasync_without_awaitrule analysis.@concurrentfunctions requiresaysncin any case.nandhinisubbu
#6283
swiftlint-static, the experimental fully-static Linux binary, now uses 512 KiBas thread stack size matching Darwin’s size to prevent stack exhaustion.
ainame
#6287
v0.61.0Compare Source
Breaking
If
SWIFTLINT_DISABLE_SOURCEKITis set to prohibit loadinglibsourcekitdInProcat runtime,rules requiring SourceKit will be disabled and a warning will be printed once per rule.
SimplyDanny
The
operator_whitespacerule has been removed and its functionality merged into thefunction_name_whitespacerule. The latter now also checks spacing around thefunckeyword, the function name/operator and its generic parameters while respecting comments.
On top of that, it supports autocorrection.
SimplyDanny
Expose only
TestHelpersandExtraRulesTestsas part of Bazel releases.SimplyDanny
Experimental
- A dynamically linked binary that requires
- A fully statically linked binary named
SimplyDannylibsourcekitdInProc.sotogether with itstransitive dependencies to be present on the system at runtime. It is named
swiftlintand the same binary as before. It supports all built-in rules.
swiftlint-staticthat does not requireany dynamic libraries at runtime. Rules requiring SourceKit will be disabled and
reported to the console when running this binary.
Enhancements
A fully statically linked Linux binary can now be built with the Swift SDK and
the compiler options
-Xswiftc -DSWIFTLINT_DISABLE_SOURCEKIT. This binary does notrequire
libsourcekitdInProc.soor any other dynamic libraries to be present on thesystem at runtime. Rules requiring SourceKit will be disabled and reported to the console
when running this binary.
SimplyDanny
Add
function_name_whitespacerule to enforce consistent spacing between thefunckeyword, function name, and its generic parameters. Ensures exactly one space between
funcand the function name, and configurable spacing around generics viageneric_spacing:no_space(default):func name<T>()leading_space:func name <T>()trailing_space:func name<T> ()leading_trailing_space:func name <T> ()Supports autocorrection.
GandaLF2006
The
operator_whitespacerule now supports autocorrection and better respects comments infunction declarations. The rule has been deprecated in favor of the new
function_name_whitespacerule. You can still refer to it by its identifier, but get checking for normal function names
on top.
SimplyDanny
In
line_lengthrule, treat strings with opening and closing quotes in the same line assingle-line strings no matter if they are enclosed by triple quotes or not. Furthermore,
improve detection of comment-only lines.
SimplyDanny
#6220
#6219
Improve detection of comment-only lines in
file_lengthrule.SimplyDanny
#6219
Rewrite
quick_discouraged_callrule with SwiftSyntax.SimplyDanny
Bug Fixes
Ensure that header matched against always end in a newline in
file_headerrule.SimplyDanny
#6227
Fix
closure_end_indentationrule reporting violations when the called baseinvolves chained optional expressions.
SimplyDanny
#6216
v0.60.0Compare Source
Breaking
Remove support for boolean values in the
validates_start_with_lowercaseoption entirely.Use the severity levels
off,warningorerrorinstead.kaseken
SwiftLint now requires macOS 13 or higher to run.
JP Simard
In
SwiftLintBuildToolPlugin, print the content of theBUILD_WORKSPACE_DIRECTORYenvironment variable only in debug builds.
SimplyDanny
#6135
The
redundant_optional_initializationrule has been replaced by theimplicit_optional_initializationrule that enforces implicit or explicit initializationof optional variables. Its default style
alwaysmimics the behavior of theredundant_optional_initializationrule by default.redundant_optional_initializationis now an alias for
implicit_optional_initialization.leo-lem
#1940
The
swiftlint_linux.ziprelease archive has been renamed toswiftlint_linux_amd64.zipto avoid confusion with the new
swiftlint_linux_arm64.ziparchive.Bradley Mackey
SimplyDanny
Experimental
Enhancements
Linting got up to 30% faster due to the praiseworthy performance
improvements done in the SwiftSyntax
library.
Ignore locally defined
countidentifiers inempty_countrule.SimplyDanny
#5326
The
private_swiftui_staterule now applies toViewModifiertypes.mt00chikin
Support for ARM64 Linux binaries has been added. The
swiftlint_linux.ziprelease archivehas been renamed to
swiftlint_linux_amd64.zip. Next to it, a newswiftlint_linux_arm64.ziparchive has been added. The
SwiftLintBinary.artifactbundle.zipnow contains both versionsof the Linux binary, so that the binary plugins
can now be used on Linux and macOS running no both AMD64 and ARM64 architectures.
Bradley Mackey
SimplyDanny
Add
include_variablesconfiguration option tonon_optional_string_data_conversionrule.When enabled, the rule will trigger on variables, properties, and function calls in addition
to string literals. Defaults to
falsefor backward compatibility.SimplyDanny
#6094
Add Sendable conformance to Rule.Type for building with Swift 6.
erikkerber
#issue_number
Fix false positives for
Actor-conforming delegate protocols in theclass_delegate_protocolrule.imsonalbajaj
#6054
Support extensions and protocols in
type_body_lengthrule. They can be configured using thenew
excluded_typesoption which by default excludesextensionandprotocoltypes.This means the rule now checks
struct,class,actorandenumby default. To enablechecking of extensions and protocols, set
excluded_typesto an empty array or exclude othertypes as needed.
SimplyDanny
Exclude explicit system modules from
duplicate_importsanalysis, that is, modulesthat are part of the system frameworks but need to be imported explicitly due to being
declared as
explicit modulein their module map.SimplyDanny
#6098
Ignore various assignment operators like
=,+=,&=, etc. with right-hand sideternary expressions otherwise violating the
void_function_in_ternaryrule.SimplyDanny
#5611
Rewrite the following rules with SwiftSyntax:
accessibility_label_for_imageaccessibility_trait_for_buttonclosure_end_indentationexpiring_todofile_headerfile_lengthline_lengthtrailing_whitespacevertical_whitespaceJP Simard
Matt Pennig
Add
excluded_pathsoption to thefile_namerule. It allows to exclude complete filepaths from analysis. All entries are treated as regular expressions. A single match in
its full path is enough to ignore a file. This is different from the
excludedoptionthat only accepts and checks against file names.
Ueeek
#6066
Fix false positives of
redundant_discardable_letrule in@ViewBuilderfunctions,#Previewmacro bodies and preview providers whenignore_swiftui_view_bodiesisenabled.
kaseken
#6063
Improve
multiline_parametersrule to correctly supportmax_number_of_single_line_parametersand detect mixed formatting.GandaLF2006
Add new
prefer_condition_listrule that triggers when aguard/if/whilecondition is composed of multiple expressions connected by the
&&operator.It suggests to use a condition list instead, which is more idiomatic.
SimplyDanny
Add
ignore_coding_keysparameter tonestingrule. Setting this to true preventsCodingKeyenums from violating the rule.braker1nine
#5641
Support deinitializers and subscripts in
function_body_lengthrule.SimplyDanny
Extend the
xct_specific_matcherrule to support identity operators (===and!==)in addition to the existing equality operators (
==and!=).SimplyDanny
#5849
Add new
implicit_optional_initializationrule to enforce implicit or explicitinitialization of optional variables, configurable via
style: always | never.It replaces the
redundant_optional_initializationrule mimicking it with thestyle: alwaysoption which is the default.leo-lem
#1940
Add new
ignore_identity_closuresparameter toprefer_key_pathsrule to skipconversion of identity closures (
{ $0 }) to identity key paths (\.self).Note that identity key paths are only supported from Swift 6 on, hence this option
will be implicitly ignored/set to
truewhen SwiftLint detects a Swift <6 compilerto avoid causing compilation errors.
p4checo
#5965
Bug Fixes
Fix
no_extension_access_modifierrule incorrectly triggering fornonisolated extension.The rule now only flags actual access control modifiers (
private,public,open,internal,fileprivate) and ignores isolation modifiers likenonisolated.copilot
#6168
Improved error reporting when SwiftLint exits, because of an invalid configuration file
or other error.
Martin Redington
#6052
Keep the default severity levels when neither
warningnorerrorvalues are configured.Ensure especially that the
errorlevel is not set tonilwhen thewarninglevelisn't set either.
SimplyDanny
v0.59.1Compare Source
Breaking
opaque_over_existentialopt-in rule as it causes too many false positivesthat can even lead to wrong-compilable code when
anyis required by a protocol thetype conforms to. In this case, using
somecan be totally valid even though theprotocol requirement is not reimplemented for the type causing behavioral changes.
SimplyDanny
Experimental
Enhancements
allowed_numbersoption to theno_magic_numbersrule.Martin Redington
Bug Fixes
v0.59.0Compare Source
Breaking
SimplyDanny
Experimental
swiftlint-devcommand line tool that's intended to help to develop SwiftLint by encapsulating repetitivetasks. It can already be used to generate templates for new rules including optional configurations and tests. Run
swift run swiftlint-dev rules template -hto get an overview of the command's usage and the available customizationoptions. To register new rules, run
swift run swiftlint-dev rules register.SimplyDanny
Enhancements
Add new option
ignores_multiline_stringstoline_lengthrule. It allows to ignore too longlines inside of multiline strings.
thisIsTheFoxe
#2689
Ignore
UIColorinitializers inno_magic_numbersrule.suojae
hyeffie
#5183
Exclude types with a
@Suiteattribute and functions annotated with@Testfromno_magic_numbersrule.Also treat a type as a
@Suiteif it contains@Testfunctions.SimplyDanny
#5964
Add new
opaque_over_existentialopt-in rule that triggers when the existentialanytype of afunction parameter can be replaced with an opaque
sometype.SimplyDanny
Add a new rationale property to rule descriptions, providing a more expansive
description of the motivation behind each rule.
Martin Redington
#5681
Add new
allowed_typesoption tolegacy_objc_typerule to ignore certain types.kapitoshka438
#3723
Add
excluded_methodsconfiguration option tounneeded_overrideto opt out checking methods with a given name.For example, this helps avoid a conflict with
balanced_xctest_lifecyclewhere one ofsetUp/tearDownis empty but the other is not.
jaredgrubb
Bug Fixes
Fix issue referencing the Tests package from another Bazel workspace.
jszumski
Fix crash when a disable command is preceded by a unicode character.
SimplyDanny
#5945
Allow severity of
duplicate_importsrule to be configurable.SimplyDanny
#5978
Silence
no_magic_numbersrule in conditions of#ifdirectives.SimplyDanny
#6000
Consider types restricting a protocol to classes in composed inherited types in
class_delegate_protocolrule.SimplyDanny
#5982
Rewrite
type_contents_orderrule with SwiftSyntax fixing an issue with experimental availability macros.SimplyDanny
#5999
Update
file_namerule to considermacrotypes.fraioli
#6026
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), 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.