Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: move prod deploy back into staged workflow #721

Merged
merged 1 commit into from
Aug 27, 2024

Conversation

MatMoore
Copy link
Contributor

We initially thought that we should trigger prod deploys on release, as this would force us to write release notes for everything we deploy.

This idea didn't work:

  1. it made it difficult to prevent skipping the earlier stages of the workflow
  2. it forces every prod deploy to be a release, even if nothing user-facing has changed

This commit moves the prod release back into the deploy-staged workflow.

To help us with release notes, it automatically generates a draft release, which we can then edit and publish. I've moved this to the very end of the workflow, to give teams reusing this code the assurance that anything we publish as a release is already running in Find MoJ data prod.

@MatMoore MatMoore requested a review from a team as a code owner August 27, 2024 13:32
@MatMoore MatMoore force-pushed the combine-prod-and-staged-workflow branch from 1bdf538 to 8997432 Compare August 27, 2024 13:34
We initially thought that we should trigger prod deploys on release, as
this would force us to write release notes for everything we deploy.

This idea didn't work:

1. it made it difficult to prevent skipping the earlier stages of the
   workflow
2. it forces every prod deploy to be a release, even if nothing
   user-facing has changed

This commit moves the prod release back into the deploy-staged workflow.

To help us with release notes, it automatically generates a draft release,
which we can then edit and publish. I've moved this to the very end
of the workflow, to give teams reusing this code the assurance that anything
we publish as a release is already running in Find MoJ data prod.
@MatMoore MatMoore force-pushed the combine-prod-and-staged-workflow branch from 8997432 to 616596f Compare August 27, 2024 13:41
Copy link
Contributor

@murdo-moj murdo-moj left a comment

Choose a reason for hiding this comment

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

Give it a test on merge :)

@MatMoore MatMoore merged commit f3aa0f2 into main Aug 27, 2024
1 check passed
@MatMoore MatMoore deleted the combine-prod-and-staged-workflow branch August 27, 2024 15:02
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