Skip to content

ci: publish package to GitHub Packages#7

Merged
BenKalsky merged 1 commit into
mainfrom
ci/publish-github-packages
May 1, 2026
Merged

ci: publish package to GitHub Packages#7
BenKalsky merged 1 commit into
mainfrom
ci/publish-github-packages

Conversation

@BenKalsky
Copy link
Copy Markdown
Member

Summary

  • Adds a manual GitHub Packages publish workflow
  • Repackages npm tarball as @digitizers/sumit-react for npm.pkg.github.com
  • Uses GITHUB_TOKEN with packages: write so the package appears in the repo sidebar

Test Plan

  • corepack pnpm test
  • corepack pnpm typecheck
  • corepack pnpm build
  • npm pack --dry-run

@BenKalsky BenKalsky merged commit 901be8e into main May 1, 2026
1 check passed
@BenKalsky BenKalsky deleted the ci/publish-github-packages branch May 1, 2026 16:00
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 855090b35a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

cache-dependency-path: |
sumit-react/pnpm-lock.yaml
sumit-api/pnpm-lock.yaml
registry-url: https://npm.pkg.github.com
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep install registry on npmjs before build steps

Setting registry-url to https://npm.pkg.github.com at setup time makes all subsequent pnpm install steps resolve packages from GitHub Packages instead of npmjs, so standard dependencies like react are looked up at npm.pkg.github.com and fail (403/404). In this workflow, installs happen immediately after setup (Build sumit-api peer package and Install sumit-react), so the publish job will fail before packaging unless the registry is switched back or scoped only to the publish step.

Useful? React with 👍 / 👎.

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.

1 participant