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
Build ci on windows #14463
Build ci on windows #14463
Conversation
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/51674/ |
640f6e0
to
11ed54d
Compare
@nicolo-ribaudo @JLHwung Now it's passable! |
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.
I like that this PR doesn't modify the windows tests to use the code compiled on windows, since our windows users will still use the code compiled&published by us on linux 👍
@@ -96,6 +96,32 @@ jobs: | |||
packages/babel-standalone/*.js | |||
!**/node_modules/** | |||
|
|||
build-windows: | |||
name: Build Babel Artifacts On Windows | |||
runs-on: windows-latest |
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.
runs-on: windows-latest | |
runs-on: windows-latest | |
needs: prepare-yarn-cache |
so that it downloads the cached packages from the previous step.
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.
Ehh, it won't restore from that step anyway. The actions/setup-node
generates different cache id from different os
https://github.com/actions/setup-node/blob/146c4d84a51256d5571b40011b76c0c6f7dfd981/src/cache-restore.ts#L39
even if Yarn 3 does support cache created from different OS.
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.
Oh ok
#14461
#14462
Need to wait for the two pull requests to be merged.
The new ci will prevent the problems of the above two pull requests.