Skip to content

NPA-6550: Fix prod deployment#127

Merged
ehallam merged 3 commits intomainfrom
task/NPA-6550/fix-prod-deployment
Apr 30, 2026
Merged

NPA-6550: Fix prod deployment#127
ehallam merged 3 commits intomainfrom
task/NPA-6550/fix-prod-deployment

Conversation

@ehallam
Copy link
Copy Markdown
Contributor

@ehallam ehallam commented Apr 30, 2026

Pull Request

🧾 Ticket Link

https://nhsd-jira.digital.nhs.uk/browse/NPA-6550


📄 Description/Summary of Changes

  • Change the GitHub environment to "production" from "prod"
  • This was an already existing bug, the environment type just never got used before

🧪 Developer Testing Carried Out


📋 PR Principles

  • Keep PRs Small and Focused: Ensure the PR addresses a single task or feature to make it easier to review.
  • Multiple PRs for one Ticket: When splitting work into multiple PRs, clearly describe what this PR addresses and outline the remaining work to complete the ticket.
  • Ensure Tests Are Included: Add or update unit, integration, or end-to-end tests to cover the changes made.
  • Follow Coding Standards: Ensure the code adheres to the team's coding guidelines and best practices.
  • Resolve Comments Promptly: If you raise a comment, ensure you follow up and resolve it before approving the PR to maintain clarity and ensure comments are addressed.
  • Foster Learning: PR reviews are an opportunity to share knowledge, provide constructive feedback, and encourage a collaborative environment.

🏷️ Naming Conventions Reminder

Please ensure the following naming conventions are followed:

  • PR title follows the format: NPA-XXXX: <short-description>
  • Branch name follows the convention: <type>/NPA-XXXX/<short-description>
  • Commit messages follow the template: NPA-XXXX: <short-description>

Copilot AI review requested due to automatic review settings April 30, 2026 10:51
@ehallam ehallam temporarily deployed to internal-dev-sandbox April 30, 2026 10:53 — with GitHub Actions Inactive
tomknapp
tomknapp previously approved these changes Apr 30, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the production deployment workflow configuration to align environment naming with the intended GitHub “production” environment, aiming to fix production deployment gating/behavior.

Changes:

  • Updated the workflow’s ENVIRONMENT value from "prod" to "production" for the setup-proxygen-prod action input.
  • Updated the deploy step environment variable ENVIRONMENT from "prod" to "production".

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/cicd-stage-3-deploy-to-prod.yml
Comment thread .github/workflows/cicd-stage-3-deploy-to-prod.yml Outdated
Comment thread .github/workflows/cicd-stage-3-deploy-to-prod.yml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/cicd-stage-3-deploy-to-prod.yml
Comment thread .github/workflows/cicd-stage-3-deploy-to-prod.yml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ehallam ehallam merged commit 2521f1d into main Apr 30, 2026
74 of 76 checks passed
@ehallam ehallam deleted the task/NPA-6550/fix-prod-deployment branch April 30, 2026 12:13
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