Skip to content

modified JS publishing workflow#1254

Open
ethankonk wants to merge 1 commit intomainfrom
ethan/js-publish-revamp
Open

modified JS publishing workflow#1254
ethankonk wants to merge 1 commit intomainfrom
ethan/js-publish-revamp

Conversation

@ethankonk
Copy link
Copy Markdown
Contributor

Summary & Motivation

  • Publish CI takes way too long, to remedy this we're cutting out versioning and changelog gen in CI and parallelizing prod/preprod tests. This cuts out a 30min job and should cut out total CI time down by ~40 mins

How I Tested These Changes

Did you add a changeset?

If updating one of our packages, you'll likely need to add a changeset to your PR. To do so, run pnpm changeset. pnpm changeset will generate a file where you should write a human friendly message about the changes. Note how this (example) includes the package name (should be auto added by the command) along with the type of semver change (major.minor.patch) (which you should set).

These changes will be used at release time to determine what packages to publish and how to bump their version. For more context see this comment.

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented Mar 25, 2026

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 18b22b0:

Sandbox Source
@turnkey/example-react-components Configuration

@ethankonk ethankonk force-pushed the ethan/js-publish-revamp branch from daa2c67 to 9b268dc Compare March 26, 2026 20:09
@ethankonk ethankonk force-pushed the ethan/js-publish-revamp branch from 9b268dc to 18b22b0 Compare March 27, 2026 16:58
Copy link
Copy Markdown
Contributor

@moeodeh3 moeodeh3 left a comment

Choose a reason for hiding this comment

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

nice I like this, but I think we should have a Makefile command written to create this release branch!

also lets not forget to update our playbook for js sdk releases after these changes 😄

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.

2 participants