Skip to content

CCM-15867 Consolidating shared scripts#502

Merged
masl2 merged 7 commits intomainfrom
CCM-15867_SharedModuleConsolidation
Mar 31, 2026
Merged

CCM-15867 Consolidating shared scripts#502
masl2 merged 7 commits intomainfrom
CCM-15867_SharedModuleConsolidation

Conversation

@aidenvaines-cgi
Copy link
Copy Markdown
Contributor

@aidenvaines-cgi aidenvaines-cgi commented Mar 27, 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

Uploading image.png…

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 27, 2026 16:25
@aidenvaines-cgi aidenvaines-cgi force-pushed the CCM-15867_SharedModuleConsolidation branch from c785420 to 08b3885 Compare March 30, 2026 14:33
@aidenvaines-cgi aidenvaines-cgi force-pushed the CCM-15867_SharedModuleConsolidation branch from 08b3885 to 03f6c70 Compare March 30, 2026 14:51
Copy link
Copy Markdown
Contributor

@stevebux stevebux left a comment

Choose a reason for hiding this comment

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

LGTM

@masl2 masl2 merged commit 22f7fb5 into main Mar 31, 2026
37 checks passed
@masl2 masl2 deleted the CCM-15867_SharedModuleConsolidation branch March 31, 2026 09:40
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.

4 participants