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

Update version of tools used #90

Merged
merged 3 commits into from
Aug 9, 2021

Conversation

kwridan
Copy link
Contributor

@kwridan kwridan commented Aug 6, 2021

Resolves: #89

Describe your changes

  • Travis image updated to Xcode12.5 (uses a newer version of homebrew)
  • SwiftFormat has been bumped to 0.48.11
  • SwiftLint has been bumped to 0.43.1
  • Code has been re-formated for compatibility with the latest SwiftFormat
    • Note a few rules have been disabled to minimize changes (those can be revisted separately if needed)
  • Added the brew installation script to the Makefile for convenience - it can be run via make install_tools

Test Plan

  • Run make install_tools
  • Verify the installation of tools works
  • Run make lint
  • Verify the checks pass

- SwiftFormat has been bumped to 0.48.11
- SwiftLint has been bumped to 0.43.1
- Code has been re-formated for compatibility with the latest SwiftFormat
  - Note a few rules have been disabled to minimize changes (those can be revisted separately if needed)
- Added the brew installation script to the Makefile for convenience - it can be run via `make install_tools`

Test Plan:

- Run `make install_tools`
- Verify the installation of tools works
- Run `make lint`
- Verify the checks pass

Signed-off-by: Kassem Wridan <[email protected]>
@codecov
Copy link

codecov bot commented Aug 6, 2021

Codecov Report

Merging #90 (31c50f0) into main (fbaf3dd) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   96.22%   96.22%           
=======================================
  Files          47       47           
  Lines        1853     1853           
=======================================
  Hits         1783     1783           
  Misses         70       70           
Impacted Files Coverage Δ
...Core/Comparator/LinkedDependenciesComparator.swift 91.66% <100.00%> (ø)
...ces/XCDiffCore/Comparator/SettingsComparator.swift 95.78% <100.00%> (ø)
...CDiffCore/Comparator/SwiftPackagesComparator.swift 96.42% <100.00%> (ø)
Sources/XCDiffCore/Library/TargetsHelper.swift 94.07% <100.00%> (ø)
Sources/XCDiffCore/Library/URL+Extensions.swift 100.00% <100.00%> (ø)

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 fbaf3dd...31c50f0. Read the comment docs.

kwridan added 2 commits August 6, 2021 14:48
Signed-off-by: Kassem Wridan <[email protected]>
@kwridan kwridan changed the title Update tools homebrew formulas Update version of tools used Aug 6, 2021
Copy link
Contributor

@marciniwanicki marciniwanicki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@kwridan kwridan merged commit dbf6929 into bloomberg:main Aug 9, 2021
@kwridan kwridan deleted the fix/update-tools-brew-formulas branch August 9, 2021 09:03
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.

Update checked in tooling brew formulas (e.g. swiftformat and swiftlint)
2 participants