Skip to content

fix(release): split express context into get-express-release-context job#8611

Merged
louib merged 1 commit into
masterfrom
vl-4192-fix-express-docker-check
Apr 23, 2026
Merged

fix(release): split express context into get-express-release-context job#8611
louib merged 1 commit into
masterfrom
vl-4192-fix-express-docker-check

Conversation

@louib
Copy link
Copy Markdown
Contributor

@louib louib commented Apr 22, 2026

Summary

  • Adds a dedicated get-express-release-context job that runs after release-bitgojs: reads the newly-bumped express version from rel/latest, resolves the lerna git tag, sanity-checks the version bump, and checks whether the Docker image already exists
  • publish-express-to-docker-hub now depends only on get-express-release-context and is skipped if docker-exists == true
  • Removes express context steps from get-release-context entirely
  • Workflow chain: get-release-contextrelease-bitgojsget-express-release-contextpublish-express-to-docker-hub

Test plan

  • Verify workflow YAML has no syntax errors
  • Trigger with dry-run: trueget-express-release-context and publish-express-to-docker-hub should be skipped
  • Trigger with dry-run: false — verify express context job reads the post-lerna version and publish job runs

Ticket: VL-4192

@linear
Copy link
Copy Markdown

linear Bot commented Apr 22, 2026

@bitgo-ai-agent-dev bitgo-ai-agent-dev Bot force-pushed the vl-4192-fix-express-docker-check branch 2 times, most recently from d85e9e5 to 64e0c9f Compare April 22, 2026 17:08
@louib louib force-pushed the vl-4192-fix-express-docker-check branch from 64e0c9f to e14f8c7 Compare April 22, 2026 17:58
@louib louib marked this pull request as ready for review April 22, 2026 17:59
@louib louib requested review from a team as code owners April 22, 2026 17:59
Extract express version/tag resolution and Docker Hub existence check
into a dedicated get-express-release-context job that runs after
release-bitgojs, so it evaluates the newly-published version rather
than the previously-released one.

Workflow chain: get-release-context → release-bitgojs
                → get-express-release-context → publish-express-to-docker-hub

Ticket: VL-4192
@louib louib force-pushed the vl-4192-fix-express-docker-check branch from e14f8c7 to c3edce2 Compare April 22, 2026 19:55
@louib louib merged commit c400d51 into master Apr 23, 2026
22 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.

3 participants