Skip to content

Update security review workflow#3333

Merged
nhatnghiho merged 2 commits into
aws:mainfrom
nhatnghiho:security-review
Jul 9, 2026
Merged

Update security review workflow#3333
nhatnghiho merged 2 commits into
aws:mainfrom
nhatnghiho:security-review

Conversation

@nhatnghiho

@nhatnghiho nhatnghiho commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Description of changes:

Update the security workflow after internal updates which introduced a new UI and addressed rate-limiting issues. The workflow will also post a comment on each PR to remind us to review the security report.

Call-outs:

PR checks will only be updated after this PR is merged, so security-review will still fail on this PR.

Testing:

New UI link

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@codecov-commenter

codecov-commenter commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.17%. Comparing base (5df320b) to head (6aa96ed).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3333      +/-   ##
==========================================
+ Coverage   78.16%   78.17%   +0.01%     
==========================================
  Files         693      693              
  Lines      123955   123955              
  Branches    17213    17211       -2     
==========================================
+ Hits        96887    96906      +19     
+ Misses      26149    26129      -20     
- Partials      919      920       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nhatnghiho nhatnghiho marked this pull request as ready for review July 6, 2026 20:28
@nhatnghiho nhatnghiho requested a review from a team as a code owner July 6, 2026 20:28
@prasden prasden self-requested a review July 7, 2026 19:08
Comment thread .github/workflows/security-review.yml
Comment thread .github/workflows/security-review.yml
Comment thread .github/workflows/security-review.yml
@nhatnghiho nhatnghiho enabled auto-merge (squash) July 8, 2026 22:26
@nhatnghiho nhatnghiho merged commit 5d82a8c into aws:main Jul 9, 2026
499 of 502 checks passed
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