Skip to content

CCM-13080: Make the docs publish run irrespective of acceptance tests#132

Closed
tdroza-nhs wants to merge 1 commit intomainfrom
feature/CCM-13080-fix-doc-deploy
Closed

CCM-13080: Make the docs publish run irrespective of acceptance tests#132
tdroza-nhs wants to merge 1 commit intomainfrom
feature/CCM-13080-fix-doc-deploy

Conversation

@tdroza-nhs
Copy link
Copy Markdown
Contributor

@tdroza-nhs tdroza-nhs commented Nov 24, 2025

Description

Docs were not being published to github because the publish step had been made to depend on acceptance-tests which don't always run. Reverting to only require the build step to have completed. Note that the publish will still only run on main as before.

Context

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.

@tdroza-nhs tdroza-nhs requested review from a team as code owners November 24, 2025 17:30
gareth-allan
gareth-allan previously approved these changes Nov 24, 2025
simonlabarere
simonlabarere previously approved these changes Nov 25, 2025
@tdroza-nhs tdroza-nhs force-pushed the feature/CCM-13080-fix-doc-deploy branch from 4dc0475 to b83fe81 Compare November 25, 2025 09:19
@tdroza-nhs tdroza-nhs enabled auto-merge (squash) November 25, 2025 09:24
@gareth-allan gareth-allan force-pushed the feature/CCM-13080-fix-doc-deploy branch from b83fe81 to 1a70abb Compare November 25, 2025 16:32
auto-merge was automatically disabled November 25, 2025 16:33

Pull request was closed

@gareth-allan gareth-allan reopened this Nov 25, 2025
@gareth-allan gareth-allan deleted the feature/CCM-13080-fix-doc-deploy branch November 25, 2025 16:39
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