-
Notifications
You must be signed in to change notification settings - Fork 89
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
Breaking changes flow #677
Conversation
8fad48e
to
4e6bb50
Compare
We should also add a new step to releasing instructions:
We can also automate this as part of the release process on CI by creating a pipeline that runs this on tag push and creates a PR. |
5b5c53f
to
56229e2
Compare
0fff810
to
3e87ad9
Compare
As I mentioned in the previous step it is not designed to be updated frequently. The script is added though. CI integration isn't important enough given that you don't need to run it frequently. But can be done in a separate PR if needed. |
4022161
to
cb01af2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just several suggestions.
And it's only 2-3 minutes to run this new CI job without a cached/vendored baseline, nice!
0a99680
to
414187e
Compare
swift package diagnose-api-breaking-changes
to detect breaking changes.