-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Fix experimental-api-diff command and add a test #3359
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
Conversation
@swift-ci please smoke test |
Hmm, I might need to skip the test in some environments - Xcode toolchains presumably won't have swift-api-digester until sometime after swiftlang/swift#36381 is merged |
0d7e47c
to
ab90dce
Compare
@swift-ci please smoke test |
Looks like in the full smoke test config on macOS we are able to run the test |
… dumping JSON first
I added a second commit here which:
|
@swift-ci please smoke test |
@swift-ci please test self hosted Linux |
@swift-ci please smoke test Linux |
@swift-ci please smoke test |
This fixes swift package experimental-api-diff, which doesn't need or accept library search path arguments. I also added a small test to avoid regressions in the short term. The motivation here is to begin working towards semantic versioning suggestions for packages when tagging a release.