Skip to content

ci: CCP-3438 draft pr testing - #382

Merged
WalterMoar merged 4 commits into
mainfrom
ci/3438-pr-deploys
Mar 2, 2026
Merged

ci: CCP-3438 draft pr testing#382
WalterMoar merged 4 commits into
mainfrom
ci/3438-pr-deploys

Conversation

@WalterMoar

@WalterMoar WalterMoar commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

Description

Update the PR pipeline to:

  • run all Analysis tests when a PR is opened, including as a draft (this allows developers to check that the tests run before marking the PR as ready for review)
  • run the OWASP ZAP Scan tests when the label zap is added to a PR (requires a deployment)

Type of change

  • ci - continuous integration and deployment pipeline

How Has This Been Tested?

The analysis tests were tested with this PR and they ran as expected when it was opened as draft. The ZAP scan can't be tested until a deployment PR is created.

Checklist

  • I have read the CONTRIBUTING doc
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have already been accepted and merged

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Comment thread .github/workflows/zap-scan.yml Fixed
@WalterMoar WalterMoar added the zap Run the OWASP ZAP Scan on this PR label Feb 26, 2026
@WalterMoar WalterMoar added zap Run the OWASP ZAP Scan on this PR and removed zap Run the OWASP ZAP Scan on this PR labels Feb 26, 2026
@WalterMoar WalterMoar changed the title ci: CCP-3438 actions workflow prettiering ci: CCP-3438 draft pr testing Feb 26, 2026
@WalterMoar WalterMoar removed the zap Run the OWASP ZAP Scan on this PR label Feb 26, 2026
@WalterMoar
WalterMoar marked this pull request as ready for review February 26, 2026 15:46
@WalterMoar
WalterMoar marked this pull request as draft February 26, 2026 15:53
@WalterMoar
WalterMoar marked this pull request as ready for review February 26, 2026 15:53
Running prettier on some files so I can create a PR to try out PRs.
@WalterMoar
WalterMoar force-pushed the ci/3438-pr-deploys branch from b721f7b to a7a2c2d Compare March 2, 2026 15:25
@WalterMoar
WalterMoar merged commit 7686465 into main Mar 2, 2026
17 checks passed
@WalterMoar
WalterMoar deleted the ci/3438-pr-deploys branch March 2, 2026 15:31
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