Skip to content

Simplify npm publish workflow#19

Closed
jozefizso wants to merge 5 commits intomainfrom
simplify-publish-workflow
Closed

Simplify npm publish workflow#19
jozefizso wants to merge 5 commits intomainfrom
simplify-publish-workflow

Conversation

@jozefizso
Copy link
Copy Markdown
Member

@jozefizso jozefizso commented May 2, 2026

Summary

  • Check both exact npm package versions with tehpsalmist/npm-publish-status-action pinned to 01cb25946b194a7a5468f22c8e74db04c283f121.
  • Start the publish job only when both action checks report exists == 0.
  • Keep the publish path small: npm ci, npm pack, GitHub artifact attestation, and npm publish --provenance.
  • Update npm publishing docs to describe the simpler release workflow.

Validation

  • Parsed .github/workflows/publish.yml with Ruby YAML.
  • Ran actionlint v1.7.12.
  • Ran git diff --check.

Leaving this PR open for maintainer review.

jozefizso and others added 5 commits May 2, 2026 13:30
## Summary
- add a release-triggered workflow that publishes @appdmg/appdmg and
@appdmg/cli
- require macOS integration tests before publishing
- pack and attest both npm tarballs, then publish those exact artifacts
with provenance enabled
- document npm Trusted Publisher settings and release order

## Verification
- ruby YAML parse for publish workflow
- npm test
- npm test --prefix packages/cli
- npm audit --audit-level=moderate
- npm audit --audit-level=moderate --prefix packages/cli
- npm ls --omit=dev --all
- npm ls --omit=dev --all --prefix packages/cli
- npm pack --dry-run
- npm pack --dry-run --prefix packages/cli
- npm run test:integration
## Summary
- skip npm publishing for GitHub prereleases so prerelease releases
cannot publish the default latest dist-tag
- check whether @appdmg/appdmg and @appdmg/cli exact versions already
exist on npm
- skip pack, attestation, artifact upload, and publish steps per package
when already published, allowing reruns after partial publish failures
- document prerelease and rerun behavior

## Verification
- YAML parse for publish workflow
@jozefizso
Copy link
Copy Markdown
Member Author

Closing after force-resetting main back before the npm publish workflow work, per maintainer request.

@jozefizso jozefizso closed this May 2, 2026
@jozefizso jozefizso deleted the simplify-publish-workflow branch May 2, 2026 12:39
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