Skip to content

CCM-15867 Consolidating shared scripts#896

Open
aidenvaines-cgi wants to merge 4 commits intomainfrom
CCM-15867_SharedModuleConsolidation
Open

CCM-15867 Consolidating shared scripts#896
aidenvaines-cgi wants to merge 4 commits intomainfrom
CCM-15867_SharedModuleConsolidation

Conversation

@aidenvaines-cgi
Copy link
Copy Markdown
Contributor

@aidenvaines-cgi aidenvaines-cgi commented Mar 31, 2026

Description

  • Bump .tool-versions ASDF and Container tools
  • Drop local repo actions for ones in shared-modules
  • Drop local pre-commit hook scripts and publish them from shared-modules

Context

NHS Notify maintains many bounded context repositories with near-identical CI workflows and pre-commit configuration. Making systemic CI/tooling changes currently requires repeated manual edits across ~20–40 repos, leading to drift, inconsistencies, and high maintenance overhead.

Type of changes

  • Refactoring (non-breaking change)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would change existing functionality)
  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I am familiar with the contributing guidelines
  • I have followed the code style of the project
  • I have added tests to cover my changes
  • I have updated the documentation accordingly
  • This PR is a result of pair or mob programming

Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

  • I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

@aidenvaines-cgi aidenvaines-cgi requested review from a team as code owners March 31, 2026 14:12
nicki-nhs
nicki-nhs previously approved these changes Apr 1, 2026
sidnhs
sidnhs previously approved these changes Apr 1, 2026
@aidenvaines-cgi aidenvaines-cgi dismissed stale reviews from sidnhs and nicki-nhs via 3fd0683 April 1, 2026 16:06
@aidenvaines-cgi aidenvaines-cgi enabled auto-merge (squash) April 1, 2026 16:06
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