-
Notifications
You must be signed in to change notification settings - Fork 8
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
v1 #540
base: main
Are you sure you want to change the base?
Conversation
|
Defining the package manager manually is no longer necessary, we should (and mostly do) use the "packageManager" property in package.json and rely on corepack to handle the rest.
Additionally, we should also rely on
.node-version
to consistently use the same version locally (with nvm or fnm) like in the CI (this needs to be migrated, but the option is there).Also refactor to use kebab-case for variable names in the setup workflow.
Breaking Changes
packageManager
input droppedpackageManagerVersion
input droppednodeVersion
input changed tonode-version
workingDirectory
input changed toworking-directory
changesets-dependencies.yaml
renamed to.github/workflows/changesets-dependencies.yml
(no "a" in extension)setupEnvWorkingDirectory
input changed tosetup-working-directory
ubuntu-latest
buildScript
TODO
v1
once this PR is merged