Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ExplicitSelfRule] Fix violation location and misplaced corrections #3507

Merged

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented Jan 29, 2021

No description provided.

@jpsim jpsim marked this pull request as ready for review January 29, 2021 17:29
@jpsim
Copy link
Collaborator Author

jpsim commented Jan 29, 2021

FYI @svenmuennich

@jpsim jpsim requested a review from keith January 29, 2021 17:29
@SwiftLintBot
Copy link

12 Messages
📖 Linting Aerial with this PR took 0.84s vs 0.75s on master (11% slower)
📖 Linting Alamofire with this PR took 1.05s vs 0.96s on master (9% slower)
📖 Linting Firefox with this PR took 4.03s vs 3.75s on master (7% slower)
📖 Linting Kickstarter with this PR took 6.35s vs 6.08s on master (4% slower)
📖 Linting Moya with this PR took 0.49s vs 0.5s on master (2% faster)
📖 Linting Nimble with this PR took 0.56s vs 0.54s on master (3% slower)
📖 Linting Quick with this PR took 0.29s vs 0.27s on master (7% slower)
📖 Linting Realm with this PR took 1.36s vs 1.31s on master (3% slower)
📖 Linting SourceKitten with this PR took 0.48s vs 0.48s on master (0% slower)
📖 Linting Sourcery with this PR took 2.78s vs 2.7s on master (2% slower)
📖 Linting Swift with this PR took 4.04s vs 3.96s on master (2% slower)
📖 Linting WordPress with this PR took 7.06s vs 6.86s on master (2% slower)

Generated by 🚫 Danger

@jpsim jpsim merged commit bbf1ad4 into master Jan 29, 2021
@jpsim jpsim deleted the explicitselfrule-fix-violation-location-and-misplaced-corrections branch January 29, 2021 18:59
DwayneCoussement pushed a commit to DwayneCoussement/SwiftLint that referenced this pull request Feb 1, 2021
commit bbf1ad4
Author: JP Simard <jp@jpsim.com>
Date:   Fri Jan 29 13:59:28 2021 -0500

    [ExplicitSelfRule] Fix violation location and misplaced corrections (realm#3507)

commit 59eb887
Author: Otavio Cordeiro <otaviocc@users.noreply.github.com>
Date:   Fri Jan 29 18:30:57 2021 +0100

    Add opt-in rule discouraged_assert (realm#3506)

    Co-authored-by: Keith Smiley <keithbsmiley@gmail.com>

commit 6a38b15
Author: JP Simard <jp@jpsim.com>
Date:   Fri Jan 22 13:31:05 2021 -0500

    Revert "Merge pull request realm#3473 from realm/feature/3472-fix-custom-rules-merging" (realm#3503)

    This reverts commit 537e53f, reversing
    changes made to ba49f7d.

commit 6de5771
Author: JP Simard <jp@jpsim.com>
Date:   Fri Jan 22 12:44:30 2021 -0500

    [Analyze] Support compile commands with relative paths (realm#3501)

    * [Analyze] Support compile commands with relative paths

    * Add changelog entry

    And made small formatting edits to other recent entries.

commit 537e53f
Merge: ba49f7d 22d25da
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Fri Jan 15 11:53:17 2021 +0100

    Merge pull request realm#3473 from realm/feature/3472-fix-custom-rules-merging

    Multiple Fixes for custom rules

commit 22d25da
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Fri Jan 15 00:14:50 2021 +0100

    Add newline at end of yml test file

commit 2d17045
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Tue Dec 22 19:41:40 2020 +0100

    [realm#3477] Fix bug that prevented the reconfiguration of a custom rule in a child config

commit a39e72e
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Mon Dec 21 12:40:53 2020 +0100

    Fix changelog

commit 56e4f17
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Sun Dec 20 00:52:17 2020 +0100

    Fix if statement formatting

commit 5bffc77
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Sun Dec 20 00:44:49 2020 +0100

    Fix custom_rules merging when a configuration is based on only_rules

commit e3e169b
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Sat Dec 19 00:11:13 2020 +0100

    Add changelog entry

commit 256288a
Author: Frederick Pietschmann <19194800+fredpi@users.noreply.github.com>
Date:   Sat Dec 19 00:03:13 2020 +0100

    Separate custom rules merging & filtering to avoid misleading warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants