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

add the build github actions workflow #20

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

rgl
Copy link
Contributor

@rgl rgl commented Oct 4, 2023

this a WIP for the github actions workflow mentioned at #17.

please do not yet merge the code until you review it, especially the TODO parts, where I request your input in order to decide how to proceed with the implementation.

to deploy this you need to:

  1. at your npmjs.com account access token page, create a new Automation Classic Token.
  2. at your github repository settings, create a new actions repository secret named NPM_TOKEN with the token created at 1.
  3. at your github repository settings, create a protected tag for the v* pattern.

to trigger a new non-release build, you need to:

  1. push code to the repository.
  2. wait for the github actions workflow to build, lint, and create the npm package (will be available as a build artifact).

to trigger a new release build and publish the npm package, you need to:

  1. push a tag that starts with a v character, e.g., v1.0.6-pre1.
  2. wait for the github actions workflow to build, lint, create the npm package (will be available as a build artifact), and publish the npm package to https://www.npmjs.com/package/whip-whep.

i've done a couple of test builds, which you can see at:

@CLAassistant
Copy link

CLAassistant commented Oct 4, 2023

CLA assistant check
All committers have signed the CLA.

@murillo128 murillo128 marked this pull request as ready for review January 19, 2024 09:50
@murillo128 murillo128 merged commit c11245e into medooze:main Jan 19, 2024
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.

3 participants