Skip to content

build: Add publishing workflows - #8

Merged
frankieyan merged 2 commits into
mainfrom
frankie/release-flow
Jan 10, 2026
Merged

frankieyan merged 2 commits into
mainfrom
frankie/release-flow

Conversation

@frankieyan

Copy link
Copy Markdown
Member

This adds the workflows for release-please and to publish the package to the GitHub packages registry.

@doistbot-app doistbot-app Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This pull request successfully introduces workflows for release automation and package publishing. However, there are a few critical issues that need to be addressed for the process to function correctly. A misconfiguration in package.json will cause publishing to fail, and the release workflow is set up to trigger the publishing job twice. Additionally, a minor improvement for stability in the publish workflow is recommended.

scope: '@doist'

- name: Ensure npm 11.5.1 or later is installed
run: npm install -g npm@latest

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[P3] Use a specific npm version range for stability.
Using npm@latest could cause this workflow to fail unexpectedly if a new major version of npm with breaking changes is released. To ensure stability and align with the step name, it's better to pin to a specific version range.

Suggested change
run: npm install -g npm@latest
run: npm install -g npm@^11.5.1

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We do the same thing in Reactist. Let's leave this for now.

Comment thread .github/workflows/release-please.yml
Comment thread package.json Outdated
@frankieyan
frankieyan force-pushed the frankie/release-flow branch from a5f2464 to 02c30d1 Compare January 10, 2026 08:36
@frankieyan
frankieyan force-pushed the frankie/release-flow branch from 02c30d1 to c9c15f4 Compare January 10, 2026 08:37
@frankieyan frankieyan added the 👀 Show PR requests review before or after merging. Can be merged once checks pass. label Jan 10, 2026
@frankieyan
frankieyan requested review from a team and gnapse and removed request for a team January 10, 2026 08:41
@frankieyan
frankieyan merged commit cb8efbd into main Jan 10, 2026
2 checks passed
@frankieyan
frankieyan deleted the frankie/release-flow branch January 10, 2026 08:42

@gnapse gnapse left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👀 Show PR requests review before or after merging. Can be merged once checks pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants