Skip to content

Conversation

@jpsim
Copy link
Collaborator

@jpsim jpsim commented Apr 22, 2023

SwiftSyntax Diff: swiftlang/swift-syntax@509.0.0-swift-5.9-DEVELOPMENT-SNAPSHOT-2023-04-10-a...509.0.0-swift-DEVELOPMENT-SNAPSHOT-2023-04-25-b

Also includes changes from #4900 to use the upstream SwiftSyntax BUILD files.

@SwiftLintBot
Copy link

SwiftLintBot commented Apr 22, 2023

2 Warnings
⚠️ If this is a user-facing change, please include a CHANGELOG entry to credit yourself!
You can find it at CHANGELOG.md.
⚠️ This PR may need tests.
17 Messages
📖 Linting Aerial with this PR took 1.1s vs 1.08s on main (1% slower)
📖 Linting Alamofire with this PR took 1.42s vs 1.4s on main (1% slower)
📖 Linting Brave with this PR took 7.69s vs 7.47s on main (2% slower)
📖 Linting DuckDuckGo with this PR took 3.68s vs 3.6s on main (2% slower)
📖 Linting Firefox with this PR took 9.49s vs 9.35s on main (1% slower)
📖 Linting Kickstarter with this PR took 10.42s vs 10.16s on main (2% slower)
📖 Linting Moya with this PR took 0.57s vs 0.56s on main (1% slower)
📖 Linting NetNewsWire with this PR took 3.2s vs 3.13s on main (2% slower)
📖 Linting Nimble with this PR took 0.66s vs 0.65s on main (1% slower)
📖 Linting PocketCasts with this PR took 7.6s vs 7.32s on main (3% slower)
📖 Linting Quick with this PR took 0.26s vs 0.25s on main (4% slower)
📖 Linting Realm with this PR took 11.86s vs 11.72s on main (1% slower)
📖 Linting Sourcery with this PR took 2.27s vs 2.23s on main (1% slower)
📖 Linting Swift with this PR took 4.89s vs 4.77s on main (2% slower)
📖 Linting VLC with this PR took 1.39s vs 1.37s on main (1% slower)
📖 Linting Wire with this PR took 8.48s vs 8.31s on main (2% slower)
📖 Linting WordPress with this PR took 11.9s vs 11.62s on main (2% slower)

Here's an example of your CHANGELOG entry:

* [SwiftSyntax] Update to latest `release/5.9` commit.  
  [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

@SimplyDanny
Copy link
Collaborator

What's wrong with SourceKitten by the way? New findings and fixes have appeared in all builds for a few days.

@jpsim
Copy link
Collaborator Author

jpsim commented Apr 23, 2023

I don't know. There might be some non-determinism in SwiftLint that's surfaced more when linting SourceKitten or maybe some CI machine has a bad checkout? A bad checkout would impact both main and PR branches though so I don't think that's it.

jpsim added 2 commits May 3, 2023 11:01
Something similar to this has been merged upstream instead now. This
also renames the repo name to SwiftSyntax in preparation for it being in
the BCR
@jpsim jpsim force-pushed the jp-update-swift-syntax-2023-04-22 branch from a662842 to 5ce9e5e Compare May 3, 2023 15:12
@jpsim jpsim changed the title [SwiftSyntax] Update to latest release/5.9 commit [SwiftSyntax] Update to latest 509.0.0 tag May 3, 2023
@jpsim jpsim marked this pull request as ready for review May 3, 2023 15:32
@jpsim
Copy link
Collaborator Author

jpsim commented May 3, 2023

Looks like this update has a small but consistent performance regression of 1-4% slower.

@jpsim jpsim merged commit 6b094dd into main May 3, 2023
@jpsim jpsim deleted the jp-update-swift-syntax-2023-04-22 branch May 3, 2023 15:34
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.

4 participants