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

chore: Add publish script #151

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

chore: Add publish script #151

wants to merge 2 commits into from

Conversation

nzakas
Copy link
Member

@nzakas nzakas commented Feb 3, 2025

Prerequisites checklist

What is the purpose of this pull request?

Add a new script to simplify publishing packages.

What changes did you make? (Give an overview)

Added a new script that automatically calculates the correct publish order for packages in this repo. It also posts to social media once complete.

All npm packages are published first, followed by all JSR packages.

This should simplify publishing going forward and allow us to simplify the release-please.yml workflow.

For now, I've added this into release-please.yml in dry run mode so it just output the commands that it would otherwise execute. I figured we can do this for a couple of releases to ensure that the commands its intends to run are correct before switching over.

Related Issues

Is there anything you'd like reviewers to focus on?

#-----------------------------------------------------------------------------

- name: Publish using new script
run: node scripts/publish.js --dry-run

Choose a reason for hiding this comment

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

you should put this in package.json and use node --run publish it's will be easier to maintain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Needs Triage
Development

Successfully merging this pull request may close these issues.

2 participants