Skip to content

Un-revert "Replace rethrows with typed throws in Graph" #692

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

Merged

Conversation

stmontgomery
Copy link
Contributor

This un-reverts the changes originally landed in #662. Those changes were reverted in #679 due to a compiler issue which broke CI builds. We believe that compiler issue has been resolved via a revert in the Swift repo, made in swiftlang/swift#76435.

Before landing this (currently draft) PR, I plan to validate that Swift's CI is no longer affected using a cross-repo CI toolchain build.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

@stmontgomery stmontgomery added enhancement New feature or request embedded-swift 📟 Embedded Swift issues labels Sep 16, 2024
@stmontgomery stmontgomery self-assigned this Sep 16, 2024
@stmontgomery
Copy link
Contributor Author

@swift-ci please test

@stmontgomery
Copy link
Contributor Author

Toolchain builds have been triggered on swiftlang/swift#76484

@grynspan grynspan added this to the Swift 6.1 milestone Sep 16, 2024
@stmontgomery stmontgomery marked this pull request as ready for review September 16, 2024 20:57
@stmontgomery stmontgomery merged commit dcc3a1f into swiftlang:main Sep 16, 2024
3 checks passed
@stmontgomery stmontgomery deleted the unrevert-graph-typed-throws branch September 16, 2024 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embedded-swift 📟 Embedded Swift issues enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants