-
Notifications
You must be signed in to change notification settings - Fork 152
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
Add support for SWIFTCI_USE_LOCAL_DEPS
convention
#311
Conversation
To use this package in utils/build-script pipeline
@swift-ci test |
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.
Thanks!
@phausler Would you mind having a new tag to pin the version including this fix in update-checkout-config.json if possible? 🙏 |
This PR has broken apps depending on swift-async-algorithms on Swift Playgrounds on iPadOS.
The reproducible example is here. Or, you can reproduce the issue by the following steps.
Then, the following error will be shown. error message
|
\apple#311 introduced `Context.environment`, so that swift-async-algorithm can no longer be used in the Swift Playgrounds. However, `Context.environment` is equivalent to `ProcessInfo.processInfo.environment`, so this is an unnecessary sacrifice. This change is intended to fix that.
The PR apple#311 introduced `Context.environment`, so that swift-async-algorithm can no longer be used in the Swift Playgrounds. However, `Context.environment` is equivalent to `ProcessInfo.processInfo.environment`, so this is an unnecessary sacrifice. This change is intended to fix that.
This package is used by https://github.com/apple/swift-sdk-generator and it needs to be built during build-script pipeline, so we need to make its dependencies follow the CI convention. CC: @MaxDesiatov