Skip to content

Sp 5786/enable publishing to npm#148

Merged
andy-little merged 9 commits into
mainfrom
SP-5786/enable-publishing-to-npm
Feb 10, 2026
Merged

Sp 5786/enable publishing to npm#148
andy-little merged 9 commits into
mainfrom
SP-5786/enable-publishing-to-npm

Conversation

@andy-little
Copy link
Copy Markdown
Contributor

Why?

We need a way to version and distribute our types and schemas so consumers don’t have to pin to commits, and so we can coordinate breaking changes. Proposal.

What?

  • adds circleci workflow to publish to npm when a release is made
  • documents the release process in the README

Here are the results from running the new CI workflow with --dry-run Results

@andy-little andy-little requested review from a team as code owners February 3, 2026 10:57
Comment thread package.json
Comment thread README.md
Comment thread README.md
## Releasing

(TODO: no good release process yet)
A single semantic change to the content tree is usually split across multiple pull requests, with Go and JS changes implemented by separate teams. It is important therefore that releases are coordinated across C&M, Spark and CP to ensure that each release contains all the changes required to support the new version of the content tree.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

praise: these are helpful docs!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

praise: 👍 thanks for the docs

adgad
adgad previously approved these changes Feb 3, 2026
Comment thread package.json
Comment thread .circleci/config.yml
Comment thread package.json Outdated
Comment thread README.md
(TODO: no good release process yet)
A single semantic change to the content tree is usually split across multiple pull requests, with Go and JS changes implemented by separate teams. It is important therefore that releases are coordinated across C&M, Spark and CP to ensure that each release contains all the changes required to support the new version of the content tree.

Before making a release you will also need to raise a pull request to increment the package.json version number, this should be made in its own PR so that at least one member of each team can approve it. Please get sign-off in the [#dynamic-storytelling-team](https://financialtimes.enterprise.slack.com/archives/C035P5DCHMH) slack channel and share this PR there. Releases are then made via GitHub Releases: they must include a SemVer tag `vX.Y.Z` which matches the one within the package json. The release notes should be completed to communicate what changed in the release. Cutting a release will trigger CI to publish to npm for JS consumers. Go consumers can depend on the new git tag as the module version.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thought (non-blocking): Would it make sense to create a dedicated channel for communicating content tree changes? I’m pretty sure the Specialists would also find it useful to follow those updates.

Comment thread README.md
## Releasing

(TODO: no good release process yet)
A single semantic change to the content tree is usually split across multiple pull requests, with Go and JS changes implemented by separate teams. It is important therefore that releases are coordinated across C&M, Spark and CP to ensure that each release contains all the changes required to support the new version of the content tree.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

praise: 👍 thanks for the docs

@epavlova epavlova requested a review from a team February 5, 2026 07:16
Copy link
Copy Markdown

@lastHokage lastHokage left a comment

Choose a reason for hiding this comment

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

Thanks Andy 🚀

@andy-little andy-little merged commit bd2e47b into main Feb 10, 2026
6 checks passed
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.

5 participants