Skip to content
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

feat(ci): add release workflow #409

Merged
merged 4 commits into from
Feb 27, 2024
Merged

feat(ci): add release workflow #409

merged 4 commits into from
Feb 27, 2024

Conversation

underfin
Copy link
Contributor

@underfin underfin commented Jan 16, 2024

Description

Here port release workflow from https://github.com/rolldown-rs/rolldown_legacy/pull/116. Here remove some platform test, only keep x86_64-apple-darwin/x86_64-pc-windows-msvc/x86_64-unknown-linux-gnu test.

The workflow is some steps at now:

  1. Run yarn verison at local,it will create new version for packages, and push it to one pr
  2. Run Release workflow action at github Action tab, choose your branch. It will trigger release build and test and publish it.

Test Plan

Here has a windows test crash, i will fix it at next days.


@underfin underfin force-pushed the chore--add-release-workflow branch from e9b8c51 to 2e938ad Compare January 16, 2024 06:17
@underfin underfin force-pushed the main branch 12 times, most recently from b0ace1b to a00c94c Compare January 17, 2024 05:55
@underfin underfin force-pushed the chore--add-release-workflow branch from 2e938ad to 50322fa Compare January 17, 2024 06:12
echo "//registry.npmjs.org/:_authToken=$NPM_TOKEN" >> ~/.npmrc
yarn lerna publish from-package --no-verify-access --yes
env:
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
Copy link
Contributor Author

@underfin underfin Jan 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here need to add npm secret to repo setting. I'm not permission with it and npm binding packages, look like @Brooooooklyn should have it, could you add it?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ROLLDOWN_NPM_TOKEN added, in case we will have more others NPM_TOKEN

@underfin underfin changed the title wip: add release workflow ci: add release workflow Jan 17, 2024
@underfin underfin changed the title ci: add release workflow feat(ci): add release workflow Jan 17, 2024
Copy link
Member

@hyf0 hyf0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to merge this if it's ready.

@underfin underfin force-pushed the chore--add-release-workflow branch from 50322fa to 6cb24ac Compare February 27, 2024 06:26
@underfin underfin merged commit 85c1b9e into main Feb 27, 2024
6 checks passed
@underfin underfin deleted the chore--add-release-workflow branch February 27, 2024 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants