Skip to content

Conversation

@ethan-ozelius-contentful
Copy link
Contributor

@ethan-ozelius-contentful ethan-ozelius-contentful commented Nov 5, 2025

Summary

Migrate from CircleCI to github action.

  1. create a new release.yml github action that includes lint, build, unit (test), integration (test) and release.
  2. Remove GITHUB_PACKAGES_WRITE_TOKEN secret injection, in favor of OIDC trusted publishing, read more, in favor of id-token: write permission
  3. Create a new prerelease channel testing-oidc-trusted-publishing to test out publishing to npmjs with trusted publishing, without polluting the main branch.
  4. set a default permission of just contents: read for the whole github action, if any individual jobs need more access, like release, they can specify.

Todos

  • Ensure that the infrastructure team has set up the trusted publisher profile from github to npmjs.

@ethan-ozelius-contentful ethan-ozelius-contentful force-pushed the testing-oidc-trusted-publishing branch from f9b4976 to e7d38fc Compare November 5, 2025 23:00
@ethan-ozelius-contentful ethan-ozelius-contentful marked this pull request as ready for review November 5, 2025 23:13
@ethan-ozelius-contentful ethan-ozelius-contentful requested a review from a team as a code owner November 5, 2025 23:13
@ethan-ozelius-contentful ethan-ozelius-contentful force-pushed the testing-oidc-trusted-publishing branch 11 times, most recently from d14f853 to f6d5201 Compare November 7, 2025 21:09
@ethan-ozelius-contentful ethan-ozelius-contentful force-pushed the testing-oidc-trusted-publishing branch 4 times, most recently from 03d2557 to 574d7b1 Compare November 7, 2025 21:45
@ethan-ozelius-contentful ethan-ozelius-contentful force-pushed the testing-oidc-trusted-publishing branch 6 times, most recently from 9896d20 to d4f9a14 Compare November 7, 2025 22:06
@elylucasctfl
Copy link
Contributor

update the PR commit when you merge to be a fix or a chore as this doesn't actually add any new functionality to the SDK

@elylucasctfl
Copy link
Contributor

Remove the circle config as well so that the CI doesn't run on circle as well

Copy link
Contributor

@elylucasctfl elylucasctfl left a comment

Choose a reason for hiding this comment

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

A few cleanup items and some updates based on some of the findings I've found going over it the past few days

@ethan-ozelius-contentful ethan-ozelius-contentful force-pushed the testing-oidc-trusted-publishing branch 5 times, most recently from 0aaa361 to 2481753 Compare November 12, 2025 22:10
@ethan-ozelius-contentful
Copy link
Contributor Author

@elylucasctfl

update the PR commit when you merge to be a fix or a chore as this doesn't actually add any new functionality to the SDK

Good catch, changed to chore

@ethan-ozelius-contentful ethan-ozelius-contentful force-pushed the testing-oidc-trusted-publishing branch 3 times, most recently from 5b58cb8 to a0ad37d Compare November 12, 2025 22:24
…pport OIDC trusted publishing.

1. Added github actions: main, check, build, notify-failure and release
2. Added a new channel testing-oidc-trusted-publishing for testing npmjs package deployments
@ethan-ozelius-contentful ethan-ozelius-contentful force-pushed the testing-oidc-trusted-publishing branch from a0ad37d to d72f880 Compare November 12, 2025 22:35
actions: read
uses: ./.github/workflows/release.yaml

notify-failure:
Copy link
Contributor

Choose a reason for hiding this comment

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

im willing to give this a shot but kinda on the fence. Reason being is our pipelines are so flakey we could get a bunch of issues (especially when dependabot drops like 10 prs at once). We'll have to monitor closely.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We basically do already with the circleCI bot.

https://contentful.slack.com/archives/C08GL0B2QT0/p1762447509895889

@ethan-ozelius-contentful ethan-ozelius-contentful merged commit 1fe7982 into master Nov 12, 2025
16 checks passed
@ethan-ozelius-contentful ethan-ozelius-contentful deleted the testing-oidc-trusted-publishing branch November 12, 2025 23:24
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