Skip to content

fix(publish): publish on Node 22; tag.js off deprecated set-output#158

Open
dselman wants to merge 1 commit into
mainfrom
ds-fix-publish-node22
Open

fix(publish): publish on Node 22; tag.js off deprecated set-output#158
dselman wants to merge 1 commit into
mainfrom
ds-fix-publish-node22

Conversation

@dselman
Copy link
Copy Markdown
Contributor

@dselman dselman commented Jun 4, 2026

Prep before cutting the v4.0.0 release.

  • npm-publish.yml builds on Node 22 (the build/deps now require >=22; markdown-transform / cicero-core 1.0.1)
  • migrate scripts/tag.js from the deprecated ::set-output to $GITHUB_OUTPUT

Note: the release should be v4.0.0 (breaking: lib/ entry, cicero-core 1.0.0+, dropped ESM). The publish builds lib/ + umd/ via prepublishOnly: build:dist.

Generated with Claude Code

…output

The build now requires Node.js 22 (engines >=22; markdown-transform / cicero-core 1.0.1),
so the publish workflow must build on 22. Also migrate scripts/tag.js from the deprecated
`::set-output` to $GITHUB_OUTPUT.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: dselman <danscode@selman.org>
@dselman dselman requested a review from a team June 4, 2026 09:20
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 26942836415

Coverage remained the same at 81.49%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 801
Covered Lines: 682
Line Coverage: 85.14%
Relevant Branches: 474
Covered Branches: 357
Branch Coverage: 75.32%
Branches in Coverage %: Yes
Coverage Strength: 5073.11 hits per line

💛 - Coveralls

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.

3 participants