Skip to content

Conversation

@alco
Copy link
Member

@alco alco commented Dec 25, 2025

I had made a few mistakes in the sync_service_dockerhub_image.yml workflow when I migrated the publishing of Docker Hub images from Buildkite to GHA:

  • forgot to replace the reference to github.event.pull_request.head.sha with the correct one
  • didn't account for the fact that GHA prevent workflows from triggering other workflows without special provisions for that

In this PR:

  • I've updated the image building workflow to check out the correct ref depending on the event that has triggered the action
  • added the option of manually triggering the action for missed releases. There's already one release that doesn't have a corresponding Docker Hub image published because of this faulty workflow
  • replaced GITHUB_TOKEN with a personal access token in changesets_release.yml, in the step where to pushes new GIt tag to GitHub. This should make the image building workflow trigerable by releases created in another workflow.

alco added 4 commits December 25, 2025 14:29
Using GITHUB_TOKEN to push new release tags to the GitHub repo prevents
triggering of other GHA actions that watch for the 'release' event.
Using a PAT instead works around this limitation.
@codecov
Copy link

codecov bot commented Dec 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.91%. Comparing base (62ef9fc) to head (29f7a8f).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3650   +/-   ##
=======================================
  Coverage   87.91%   87.91%           
=======================================
  Files          22       22           
  Lines        1820     1820           
  Branches      458      460    +2     
=======================================
  Hits         1600     1600           
  Misses        218      218           
  Partials        2        2           
Flag Coverage Δ
packages/experimental 87.73% <ø> (ø)
packages/react-hooks 86.48% <ø> (ø)
packages/start 89.58% <ø> (ø)
packages/typescript-client 93.66% <ø> (ø)
packages/y-electric 56.05% <ø> (ø)
typescript 87.91% <ø> (ø)
unit-tests 87.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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