Skip to content

[chores] Add process for first-time package publishing#2055

Merged
Drarig29 merged 31 commits intomasterfrom
corentin.girard/SYNTH-23939/init-plugin
Jan 19, 2026
Merged

[chores] Add process for first-time package publishing#2055
Drarig29 merged 31 commits intomasterfrom
corentin.girard/SYNTH-23939/init-plugin

Conversation

@Drarig29
Copy link
Contributor

@Drarig29 Drarig29 commented Jan 15, 2026

What and why?

This PR adds a yarn check-npm-packages --fix script to publish an initial version for every new package, so that we can enable OIDC on NPM side for their next releases.

How?

  • In CI, use yarn check-npm-packages without the --fix argument
    • This only happens during the release process to reduce friction to non-admin contributors
    • If the check fails:
      • The CI on the release PR fails
      • The author of the release PR is pinged in a PR comment telling them to ask an admin
      • The release process is paused until an admin follows the instructions here

[Outdated but still relevant] I tested this PR with our future @datadog/datadog-ci-plugin-junit package:

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)

@Drarig29 Drarig29 added the chores Related to the CI or developer experience label Jan 15, 2026
@Drarig29 Drarig29 force-pushed the corentin.girard/SYNTH-23939/init-plugin branch from d2e20b5 to 1b58d6f Compare January 15, 2026 16:31
Comment on lines +261 to +262
node-version: '24' # Needed for OIDC publishing
registry-url: 'https://registry.npmjs.org'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

- run: yarn install --immutable
- run: yarn build
- run: yarn publish:all
- run: yarn publish:all --provenance
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reintroduced --provenance to get back the checkmark on NPM:

Image

@Drarig29 Drarig29 marked this pull request as ready for review January 15, 2026 19:17
@Drarig29 Drarig29 requested review from a team as code owners January 15, 2026 19:17
@Drarig29 Drarig29 force-pushed the corentin.girard/SYNTH-23939/init-plugin branch from ce170b7 to aefdd28 Compare January 15, 2026 20:52
@Drarig29
Copy link
Contributor Author

Drarig29 commented Jan 15, 2026

Failing as expected:

image

I'm now going to base this PR off of #2056.

@Drarig29 Drarig29 requested review from a team as code owners January 15, 2026 20:55
@Drarig29 Drarig29 requested a review from rodrimendoza January 15, 2026 20:55
@Drarig29 Drarig29 changed the base branch from master to corentin.girard/SYNTH-21709/migrate-junit January 15, 2026 20:56
@Drarig29 Drarig29 removed request for a team and rodrimendoza January 15, 2026 20:56
Copy link
Contributor

@ava-silver ava-silver left a comment

Choose a reason for hiding this comment

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

one nit, but overall lgtm!

Base automatically changed from corentin.girard/SYNTH-21709/migrate-junit to master January 19, 2026 10:34
Comment on lines +25 to +29
- name: Check NPM packages
run: yarn check-npm-packages
env:
# Used to post comments on the PR
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Example comment: #2061 (comment)

Image

@Drarig29 Drarig29 changed the title [chores] Add bin/init-package.sh script [chores] Add process for first-time package publish Jan 19, 2026
@Drarig29 Drarig29 changed the title [chores] Add process for first-time package publish [chores] Add process for first-time package publishing Jan 19, 2026
@Drarig29 Drarig29 merged commit 030c66c into master Jan 19, 2026
26 checks passed
@Drarig29 Drarig29 deleted the corentin.girard/SYNTH-23939/init-plugin branch January 19, 2026 13:10
@Drarig29 Drarig29 mentioned this pull request Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chores Related to the CI or developer experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants