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

Change links to new location #86

Merged
merged 2 commits into from
Jul 10, 2024
Merged

Conversation

SwiftCoderJoe
Copy link
Contributor

@SwiftCoderJoe SwiftCoderJoe commented Jul 9, 2024

Summary

This fixes some 404s and dead links, especially with documentation. It's also necessary to change the project link in the GitHub settings, but that obviously must be done by a maintainer.

Testing

I tried to make sure I didn't miss any links and didn't break any existing ones, but it's worth a check.

This fixes some 404s and dead links
Copy link
Contributor

@d-ronnqvist d-ronnqvist left a comment

Choose a reason for hiding this comment

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

This looks good to me. Thanks for making these changes. Just remove the .vscode/launch.json file and this will be ready to merge.

.vscode/launch.json Outdated Show resolved Hide resolved
Package.swift Show resolved Hide resolved
Copy link
Contributor Author

@SwiftCoderJoe SwiftCoderJoe left a comment

Choose a reason for hiding this comment

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

Made the changes you requested. Let me know if you still want me to change package.swift

@d-ronnqvist
Copy link
Contributor

Made the changes you requested. Let me know if you still want me to change package.swift

Thank you. Like I said, I think this change to the Package.swift file is fine since there's only one dependency and it has moved to a new organization.

Copy link
Contributor

@d-ronnqvist d-ronnqvist left a comment

Choose a reason for hiding this comment

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

LGTM

@d-ronnqvist
Copy link
Contributor

@swift-ci please test

@d-ronnqvist d-ronnqvist merged commit 5761ba9 into swiftlang:main Jul 10, 2024
2 checks passed
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