-
Notifications
You must be signed in to change notification settings - Fork 155
Perception 2.0 #296
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
Perception 2.0 #296
Conversation
@gemmakbarlow Just a heads up but when working on this PR SwiftSyntax v601 got pulled in temporarily, which is not library evolution-compatible, so I think our library evolution compatibility is a little tenuous and doesn't reflect the experience folks may have when trying to use our library. Not sure if you'd want to look into contributing a fix upstream to SwiftSyntax. Also, while I was able to restore CI to an earlier pin of SwiftSyntax, we may have to disable library evolution CI on repos like this one in the future. We definitely aspire to make things as compatible as possible, but we unfortunately don't have the bandwidth to ensure that more edge case CI stays green. |
No worries, feel free to revert anything I added there. I probably don't have the bandwidth for the upstream fix either at the minute, but will take a look. |
@gemmakbarlow Looks like it failed to pin after all 😕. I'm going to disable CI for now but whenever things are fixed in SwiftSyntax we will happily re-enable! |
Prep for the release of pointfreeco/swift-perception#141