Update dependency realm/SwiftLint to v0.52.2 #20
Merged
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.51.0
->0.52.2
Release Notes
realm/SwiftLint
v0.52.2
Compare Source
Breaking
Experimental
Enhancements
self.x = x
from being reported bythe
redundant_self_in_closure
rule.SimplyDanny
#4988
Bug Fixes
Make
unhandled_throwing_task
opt-in instead of enabled by default. The ruleis still prone to false positives at this point, so this makes enabling the
rule a conscious decision by end-users.
JP Simard
#4987
Fix
unhandled_throwing_task
false positives when theTask
is returned orwhere the throwing code is handled in a
Result
initializer.JP Simard
#4987
v0.52.1
Compare Source
Breaking
Experimental
Enhancements
Bug Fixes
Let the
validates_start_with_lowercase
option in name configurationsexpect a severity (warning or error). Not setting it disables the check.
Boolean values are now deprecated. A
true
value enables the check as anerror for the time being to keep the previous behavior.
SimplyDanny
#2180
Fixed a false positive in
unhandled_throwing_task
.kylebshr
#4984
Fix Bazel release tarball for compiling on macOS.
JP Simard
#4985
v0.52.0
Compare Source
Breaking
The
attributes
rule now expects attributes with arguments to be placedon their own line above the declaration they are supposed to influence.
This applies to attributes with any kinds of arguments including single
key path arguments which were previously handled in a different way. This
behavior can be turned off by setting
attributes_with_arguments_always_on_line_above
to `false.
SimplyDanny
#4843
The internal module structure for SwiftLint has changed to split the
monolithic
SwiftLintFramework
into newSwiftLintCore
for core linterinfrastructure,
SwiftLintBuiltInRules
for built-in rules andSwiftLintExtraRules
to add your own native rules to SwiftLint.JP Simard
Experimental
Enhancements
Add new
superfluous_else
rule that triggers onif
-statements when anattached
else
-block can be removed, because all branches of the previousif
-block(s) would certainly exit the current scope already.SimplyDanny
Add
sorted_enum_cases
rule which warns when enum cases are not sorted.kimdv
Add new
redundant_self_in_closure
rule that triggers in closures onexplicitly used
self
when it's actually not needed due to:self
({ [self] in ... }
)self
can always be omitted){ ... }()
) as they aredefinitly not escaping
self
with explicit unwrappingSimplyDanny
#59
Extend
xct_specific_matcher
rule to check for boolean asserts on (un)equalcomparisons. The rule can be configured with the matchers that should trigger
rule violations. By default, all matchers trigger, but that can be limited to
just
one-argument-asserts
ortwo-argument-asserts
.SimplyDanny
JP Simard
#3726
Trigger
prefer_self_in_static_references
rule on more type references.SimplyDanny
Adds a new
reporters
command, to improve discoverability of reporters.Martin Redington
#4819
Adds
test_parent_classes
option to theno_magic_numbers
rule.Violations within test classes will now be ignored by default.
Martin Redington
#4896
Stop enforcing calls to super from the override functions
setUp()
,tearDown()
,setUpWithError()
, andtearDownWithError()
inXCTestCase
subclasses.
AndrewDMontgomery
#4875
Prepend
warning:
to error messages so that they show in Xcode.whiteio
#4923
The
attributes
rule received a new boolean optionattributes_with_arguments_always_on_line_above
which istrue
by default.Setting it to
false
ensures that attributes with arguments like@Persisted(primaryKey: true)
don't violate the rule if they are on the sameline with the variable declaration.
SimplyDanny
#4843
Add new
unhandled_throwing_task
rule that triggers when a Task with animplicit error type has unhandled trys or errors thrown inside its body.
This results in errors being silently discarded, which may be unexpected.
See this forum thread for more details: https://forums.swift.org/t/56066\
kylebshr
Bug Fixes
Fix
lower_acl_than_parent
rule rewriter by preserving leading whitespace.SimplyDanny
#4860
Ignore block comments in
let_var_whitespace
rule.SimplyDanny
#4871
Fix false positives in
indentation_width
rule.Sven Münnich
Do not trigger
reduce_boolean
onreduce
methods with a first namedargument that is different from
into
.SimplyDanny
#4894
Work around dyld warning about duplicate SwiftSyntax classes.
keith
#4782
Improve lint times of SwiftLintPlugin by moving the
excludedPaths(fileManager:)
operation out of the linting iterations.andyyhope
#4844
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 has been generated by Mend Renovate. View repository job log here.