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

Automate publishing process to avoid issues #280

Open
giordano opened this issue Aug 30, 2024 · 1 comment · May be fixed by #281
Open

Automate publishing process to avoid issues #280

giordano opened this issue Aug 30, 2024 · 1 comment · May be fixed by #281

Comments

@giordano
Copy link
Member

Feels like all the instructions in that file should be a dispatch workflow so that you click a button and everything is done automatically.

Originally posted by @giordano in #278 (comment)

@DilumAluthge
Copy link
Member

Alternate proposal:

  1. Switch to the master branch of this repo being deployable (i.e. dist/index.js is checked into the repo on the master branch).
  2. Then, to make a new release of this repo, you just go to the Releases tab, and create a new release off of the latest commit on master.
  3. Also, remove the version field from package.json and package-lock.json.

An added benefit of the above approach is that all PR branches will become usable, so if someone wants to try out a PR before it is merge, you just point to the PR branch.

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 a pull request may close this issue.

2 participants