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

[6.0] Update links for repositories moved to the swiftlang org on GitHub #758

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Jun 25, 2024

No description provided.

@ahoppen ahoppen requested review from allevato and bnbarham June 25, 2024 14:03
CMakeLists.txt Outdated
@@ -68,7 +68,7 @@ endif()
find_package(SwiftSyntax CONFIG)
if(NOT SwiftSyntax_FOUND)
FetchContent_Declare(Syntax
GIT_REPOSITORY https://github.com/apple/swift-syntax
GIT_REPOSITORY https://github.com/swiftlang/swift-syntax
GIT_TAG main)
Copy link
Member

Choose a reason for hiding this comment

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

Unrelated to this PR, but I just noticed that CMake is referring to the main branch here. Does that need to be fixed on the 6.0 branch?

Copy link
Contributor

Choose a reason for hiding this comment

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

Heh, probably. It wouldn't be used in CI at all. @compnerd what if instead of specifying repository + tag, we just specify a path that's passed down and then rely on update-checkout? It makes it a little more clunky to eg. just build the repo itself, but is maybe better than having yet another place to update versions?

Copy link
Member Author

Choose a reason for hiding this comment

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

I updated the branch to release/6.0 for now. But yes, would be nice to not have yet another place that needs version updates.

@ahoppen ahoppen force-pushed the 6.0/swiftlang-migration branch from 7ca401c to 7200769 Compare June 26, 2024 04:23
@ahoppen ahoppen merged commit 9f89186 into swiftlang:release/6.0 Jun 26, 2024
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