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 SwiftSyntax to 0.50600.0 #3901

Merged
merged 9 commits into from
Mar 17, 2022
Merged

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented Mar 16, 2022

Uses SwiftSyntax 5.5 on Linux when building with Swift 5.5. We use the 5.6 version of SwiftSyntax when building with Swift 5.5 and 5.6 on macOS because we statically link lib_InternalSwiftSyntaxParser thanks to https://github.com/keith/StaticInternalSwiftSyntaxParser/releases/tag/5.6.

This keeps SwiftLint binaries portable across machines on macOS, regardless of where or even if lib_InternalSwiftSyntaxParser is installed.

@jpsim jpsim force-pushed the update-swiftsyntax-to-0.50600.0 branch from 7e400e0 to b60cf2d Compare March 16, 2022 16:15
@SwiftLintBot
Copy link

SwiftLintBot commented Mar 16, 2022

2 Warnings
⚠️ Please include a CHANGELOG entry to credit yourself!
You can find it at CHANGELOG.md.
⚠️ This PR may need tests.
12 Messages
📖 Linting Aerial with this PR took 0.93s vs 0.92s on master (1% slower)
📖 Linting Alamofire with this PR took 1.07s vs 1.06s on master (0% slower)
📖 Linting Firefox with this PR took 3.9s vs 3.85s on master (1% slower)
📖 Linting Kickstarter with this PR took 7.03s vs 7.01s on master (0% slower)
📖 Linting Moya with this PR took 4.35s vs 4.36s on master (0% faster)
📖 Linting Nimble with this PR took 0.38s vs 0.38s on master (0% slower)
📖 Linting Quick with this PR took 0.17s vs 0.16s on master (6% slower)
📖 Linting Realm with this PR took 5.95s vs 5.94s on master (0% slower)
📖 Linting SourceKitten with this PR took 0.32s vs 0.32s on master (0% slower)
📖 Linting Sourcery with this PR took 1.83s vs 1.84s on master (0% faster)
📖 Linting Swift with this PR took 3.04s vs 3.05s on master (0% faster)
📖 Linting WordPress with this PR took 7.12s vs 7.11s on master (0% slower)

Here's an example of your CHANGELOG entry:

* Update SwiftSyntax to 0.50600.0.  
  [jpsim](https://github.com/jpsim)
  [#issue_number](https://github.com/realm/SwiftLint/issues/issue_number)

note: There are two invisible spaces after the entry's text.

Generated by 🚫 Danger

@codecov-commenter
Copy link

codecov-commenter commented Mar 16, 2022

Codecov Report

Merging #3901 (e1cd7c8) into master (ca2712a) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3901      +/-   ##
==========================================
+ Coverage   92.60%   92.61%   +0.01%     
==========================================
  Files         436      436              
  Lines       21964    21964              
==========================================
+ Hits        20339    20342       +3     
+ Misses       1625     1622       -3     
Impacted Files Coverage Δ
...LintFramework/Extensions/SwiftLintFile+Cache.swift 84.66% <ø> (ø)
Source/SwiftLintFramework/Models/Linter.swift 80.79% <0.00%> (+1.69%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@jpsim
Copy link
Collaborator Author

jpsim commented Mar 17, 2022

Reported the TSan error here: https://bugs.swift.org/browse/SR-16004

@jpsim jpsim marked this pull request as ready for review March 17, 2022 12:58
This actually works now.
@jpsim jpsim merged commit de3569c into master Mar 17, 2022
@jpsim jpsim deleted the update-swiftsyntax-to-0.50600.0 branch March 17, 2022 17:19
jpsim added a commit that referenced this pull request Mar 17, 2022
Uses SwiftSyntax 5.5 on Linux when building with Swift 5.5. We use the 5.6 version of
SwiftSyntax when building with Swift 5.5 and 5.6 on macOS because we statically link
`lib_InternalSwiftSyntaxParser` thanks to
https://github.com/keith/StaticInternalSwiftSyntaxParser/releases/tag/5.6.

This keeps SwiftLint binaries portable across machines on macOS, regardless of
_where_ or even _if_ `lib_InternalSwiftSyntaxParser` is installed.

* Run TSan CI job with `--configuration release` to avoid stack overflows
* Add Swift 5.6 CI job
* Fix linker settings
coffmark pushed a commit to coffmark/SwiftLint that referenced this pull request Apr 11, 2022
Uses SwiftSyntax 5.5 on Linux when building with Swift 5.5. We use the 5.6 version of
SwiftSyntax when building with Swift 5.5 and 5.6 on macOS because we statically link
`lib_InternalSwiftSyntaxParser` thanks to
https://github.com/keith/StaticInternalSwiftSyntaxParser/releases/tag/5.6.

This keeps SwiftLint binaries portable across machines on macOS, regardless of
_where_ or even _if_ `lib_InternalSwiftSyntaxParser` is installed.

* Run TSan CI job with `--configuration release` to avoid stack overflows
* Add Swift 5.6 CI job
* Fix linker settings
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