Skip to content

Update SwiftLint configuration#154

Merged
mattrubin merged 7 commits into
developfrom
lint
Dec 22, 2017
Merged

Update SwiftLint configuration#154
mattrubin merged 7 commits into
developfrom
lint

Conversation

@mattrubin

Copy link
Copy Markdown
Owner
  • Enable several new opt-in rules for SwiftLint 0.24
  • Remove explicit opt-in for the enabled-by-default vertical_whitespace rule
  • Re-enable the syntactic_sugar rule
  • Use the default line length warning threshold

@codecov

codecov Bot commented Dec 22, 2017

Copy link
Copy Markdown

Codecov Report

Merging #154 into develop will increase coverage by 1.09%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #154      +/-   ##
===========================================
+ Coverage    94.27%   95.36%   +1.09%     
===========================================
  Files            6        6              
  Lines          384      475      +91     
  Branches         0       33      +33     
===========================================
+ Hits           362      453      +91     
  Misses          22       22
Impacted Files Coverage Δ
Sources/Token+URL.swift 97.46% <100%> (+0.56%) ⬆️
Sources/Token.swift 100% <0%> (ø) ⬆️
Sources/Crypto.swift 100% <0%> (ø) ⬆️
Sources/PersistentToken.swift 100% <0%> (ø) ⬆️
Sources/Generator.swift 100% <0%> (ø) ⬆️
Sources/Keychain.swift 87.23% <0%> (+3.16%) ⬆️

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 bc5d910...0439077. Read the comment docs.

@mattrubin mattrubin merged commit be70abd into develop Dec 22, 2017
@mattrubin mattrubin deleted the lint branch December 22, 2017 15:27
@mattrubin mattrubin added this to the 3.1 milestone Mar 17, 2018
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.

1 participant