Skip to content

Update the pipeline configuration so it passes secrets rather than inheriting it#462

Merged
mrlockstar merged 1 commit intomainfrom
LCS-867-Review-sonarqube-vulnerabilities-in-github-pipelines-part3
May 8, 2026
Merged

Update the pipeline configuration so it passes secrets rather than inheriting it#462
mrlockstar merged 1 commit intomainfrom
LCS-867-Review-sonarqube-vulnerabilities-in-github-pipelines-part3

Conversation

@mrlockstar
Copy link
Copy Markdown
Contributor

@mrlockstar mrlockstar commented May 8, 2026

What is the change?

I forgot to add this into one of the pipeline files. It breaks the CI/CD pipeline, this should fix it

Why are we making this change?

A part of the plan to make the repo more secure and to increase the security rating of the repo.

@mrlockstar mrlockstar force-pushed the LCS-867-Review-sonarqube-vulnerabilities-in-github-pipelines-part3 branch from 8768e95 to 4966f6a Compare May 8, 2026 11:40
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Alastair Lock <alastair.lock@gmail.com>
@mrlockstar mrlockstar force-pushed the LCS-867-Review-sonarqube-vulnerabilities-in-github-pipelines-part3 branch from 4966f6a to e8792a0 Compare May 8, 2026 11:50
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

@mrlockstar mrlockstar changed the title Apply suggestions from code review Update the pipeline configuration so it passes secrets rather than inherit it May 8, 2026
@mrlockstar mrlockstar changed the title Update the pipeline configuration so it passes secrets rather than inherit it Update the pipeline configuration so it passes secrets rather than inheriting it May 8, 2026
@mrlockstar mrlockstar enabled auto-merge May 8, 2026 11:55
@mrlockstar mrlockstar merged commit 3340bfe into main May 8, 2026
27 checks passed
@mrlockstar mrlockstar deleted the LCS-867-Review-sonarqube-vulnerabilities-in-github-pipelines-part3 branch May 8, 2026 11:58
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