Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix mark rule corrections generating invalid code in some cases #1706

Merged
merged 1 commit into from
Jul 23, 2017
Merged

Fix mark rule corrections generating invalid code in some cases #1706

merged 1 commit into from
Jul 23, 2017

Conversation

marcelofabri
Copy link
Collaborator

@marcelofabri marcelofabri commented Jul 23, 2017

Apparently, we never invalidated the file cache when autocorrecting 😬

Fixes #1029 (finally!).

Apparently, we never invalidated the file cache when autocorrecting 😬
@SwiftLintBot
Copy link

12 Messages
πŸ“– Linting Aerial with this PR took 0.36s vs 0.33s on master (9% slower)
πŸ“– Linting Alamofire with this PR took 2.22s vs 2.23s on master (0% faster)
πŸ“– Linting Firefox with this PR took 9.65s vs 9.73s on master (0% faster)
πŸ“– Linting Kickstarter with this PR took 14.1s vs 13.85s on master (1% slower)
πŸ“– Linting Moya with this PR took 0.66s vs 0.66s on master (0% slower)
πŸ“– Linting Nimble with this PR took 1.28s vs 1.28s on master (0% slower)
πŸ“– Linting Quick with this PR took 0.42s vs 0.42s on master (0% slower)
πŸ“– Linting Realm with this PR took 1.96s vs 1.94s on master (1% slower)
πŸ“– Linting SourceKitten with this PR took 0.75s vs 0.75s on master (0% slower)
πŸ“– Linting Sourcery with this PR took 2.67s vs 2.66s on master (0% slower)
πŸ“– Linting Swift with this PR took 9.6s vs 9.52s on master (0% slower)
πŸ“– Linting WordPress with this PR took 8.92s vs 9.06s on master (1% faster)

Generated by 🚫 Danger

@codecov-io
Copy link

codecov-io commented Jul 23, 2017

Codecov Report

Merging #1706 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1706      +/-   ##
=========================================
+ Coverage   87.37%   87.4%   +0.02%     
=========================================
  Files         205     205              
  Lines       10146   10146              
=========================================
+ Hits         8865    8868       +3     
+ Misses       1281    1278       -3
Impacted Files Coverage Ξ”
Source/SwiftLintFramework/Rules/MarkRule.swift 100% <100%> (ΓΈ) ⬆️
Tests/SwiftLintFrameworkTests/RulesTests.swift 100% <100%> (ΓΈ) ⬆️
...SwiftLintFramework/Extensions/File+SwiftLint.swift 93.2% <100%> (+0.03%) ⬆️
...rce/SwiftLintFramework/Extensions/File+Cache.swift 77.77% <100%> (-0.21%) ⬇️
Source/SwiftLintFramework/Models/Correction.swift 60% <0%> (+60%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data
Powered by Codecov. Last update bb69302...374731d. Read the comment docs.

@marcelofabri marcelofabri merged commit aac7ca8 into realm:master Jul 23, 2017
@marcelofabri marcelofabri deleted the bugfix-1029 branch July 23, 2017 01:25
@jpsim
Copy link
Collaborator

jpsim commented Jul 25, 2017

Nice catch, nice fix, πŸ‘ , just a typo: issue1029Corretion, fixed in 0c5d331

@marcelofabri
Copy link
Collaborator Author

nice catch on that typo πŸ•΅οΈ

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants