Skip to content

Revert "APM-6800"#436

Merged
sathiya-nhs merged 1 commit intomasterfrom
revert-434-APM-6800-Dependabotfix
Jan 29, 2026
Merged

Revert "APM-6800"#436
sathiya-nhs merged 1 commit intomasterfrom
revert-434-APM-6800-Dependabotfix

Conversation

@sathiya-nhs
Copy link
Contributor

Reverts #434

@github-actions
Copy link

This branch is work on a ticket in the NHS Digital AMB JIRA Project. Here's a handy link to the ticket:

APM-6800

@github-actions
Copy link

This branch is working on a ticket to deploy to internal-portal. Here's a handy link to the specification:

revert-434-APM-6800-Dependabotfix specification

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

permissions:
contents: read
push
permissions: read-all

Check notice

Code scanning / SonarCloud

Read-all and Write-all permissions should not be used Low

Replace "read-all" with specific permissions (e.g., "contents: read"). See more on SonarQube Cloud

run: echo ::set-env name=SPEC_VERSION::$(poetry run python scripts/calculate_version.py)
env:
ACTIONS_ALLOW_UNSECURE_COMMANDS: true

Check notice

Code scanning / SonarCloud

ACTIONS_ALLOW_UNSECURE_COMMANDS should not be used Low

Remove this `ACTIONS_ALLOW_UNSECURE_COMMANDS` declaration. See more on SonarQube Cloud
push
permissions:
contents: read
permissions: read-all

Check notice

Code scanning / SonarCloud

Read-all and Write-all permissions should not be used Low

Replace "read-all" with specific permissions (e.g., "contents: read"). See more on SonarQube Cloud
@sathiya-nhs sathiya-nhs merged commit d209915 into master Jan 29, 2026
4 of 6 checks passed
@sathiya-nhs sathiya-nhs deleted the revert-434-APM-6800-Dependabotfix branch January 30, 2026 14:01
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