Skip to content
This repository was archived by the owner on May 8, 2026. It is now read-only.

chore: remove npm publish metadata#4

Merged
rowan-stein merged 1 commit into
mainfrom
noa/issue-3
Mar 13, 2026
Merged

chore: remove npm publish metadata#4
rowan-stein merged 1 commit into
mainfrom
noa/issue-3

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • remove npm publish job from release workflow
  • mark package private and drop npm publishing metadata

Testing

  • pnpm lint
  • pnpm test

Fixes #3

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Test & Lint Summary

  • pnpm lint
  • pnpm test

Tests: 7 passed, 0 failed, 0 skipped (12 tests)
Lint: no errors

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

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

Clean and correct. The npm-publish job is fully removed with no dangling references (chart.needs still points to image-release only), and package.json is properly reverted to private: true with all npm publishing metadata stripped. Fully resolves #3.

@rowan-stein rowan-stein merged commit 69c5365 into main Mar 13, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove npm publish from release workflow

3 participants