Skip to content

Add workflow summary links to PR comments - #215

Merged
felickz merged 6 commits into
mainfrom
copilot/add-context-to-pr-comments
Aug 23, 2026
Merged

Add workflow summary links to PR comments#215
felickz merged 6 commits into
mainfrom
copilot/add-context-to-pr-comments

Conversation

Copilot AI commented Aug 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Policy violation comments show counts without a direct path to remediation details. This change links them to the originating workflow run summary.

Changes

  • Builds the run URL from GitHub Actions context, including GHES support.
  • Adds the link to new and updated PR comments.
  • Omits the link when run context is incomplete.
[View workflow run summary](https://github.example.com/owner/repo/actions/runs/123456)

Copilot AI lite review requested due to automatic review settings August 23, 2026 01:28
Copilot AI reviewed Aug 23, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Copilot AI linked an issue Aug 23, 2026 that may be closed by this pull request
Add more context to PR comments #67
Closed
@felickz
Add workflow summary link to PR comments
ff526d6 
Co-authored-by: felickz <1760475+felickz@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 23, 2026 01:32
Copilot AI reviewed Aug 23, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread ghascompliance/octokit/pullrequest.py Outdated
Copilot AI changed the title [WIP] Add more context to PR comments about violations Add workflow summary links to PR comments Aug 23, 2026
Copilot AI requested a review from felickz August 23, 2026 01:35
@felickz
felickz marked this pull request as ready for review August 23, 2026 01:39
@felickz
felickz requested a review from a team as a code owner August 23, 2026 01:39
@felickz
felickz requested a review from adrienpessu August 23, 2026 01:39
@github-actions

github-actions Bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@felickz
Normalise workflow run URL env values and rename variable
d77c73f 
Co-authored-by: felickz <1760475+felickz@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 23, 2026 01:40
@felickz

felickz commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

@copilot fix the failing CI (https://github.com/advanced-security/policy-as-code/actions/runs/32610917548/job/97123834721?pr=215 https://github.com/advanced-security/policy-as-code/actions/runs/32610917548/job/97123834634?pr=215 https://github.com/advanced-security/policy-as-code/actions/runs/32610917548/job/97123834749?pr=215 https://github.com/advanced-security/policy-as-code/actions/runs/32610917548/job/97123834742?pr=215) for this PR

Copilot AI reviewed Aug 23, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread tests/test_octokit.py
@felickz
Add unit test for updatePullRequestComment link path
20ef222 
Co-authored-by: felickz <1760475+felickz@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 23, 2026 01:44
@felickz
Fix black formatting and add update-comment link test
e22788b 
Co-authored-by: felickz <1760475+felickz@users.noreply.github.com>
@felickz
Remove duplicate update-comment link test
c0354da 
Co-authored-by: felickz <1760475+felickz@users.noreply.github.com>

Copilot AI commented Aug 23, 2026

Copy link
Copy Markdown
Contributor Author

@copilot fix the failing CI (https://github.com/advanced-security/policy-as-code/actions/runs/32610917548/job/97123834721?pr=215 https://github.com/advanced-security/policy-as-code/actions/runs/326109...

Fixed. The failure was black formatting on tests/test_octokit.py (introduced in d77c73f). Reformatted the file and confirmed pipenv run lint and the full test suite (53 tests) pass locally.

Copilot AI reviewed Aug 23, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Copilot AI review requested due to automatic review settings August 23, 2026 01:46
@felickz
felickz merged commit 8a10890 into main Aug 23, 2026
18 checks passed
@felickz
felickz deleted the copilot/add-context-to-pr-comments branch August 23, 2026 01:46
Copilot AI reviewed Aug 23, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@felickz felickz Awaiting requested review from felickz
@adrienpessu adrienpessu Awaiting requested review from adrienpessu adrienpessu is a code owner automatically assigned from advanced-security/oss-maintainers

Assignees

@felickz felickz

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add more context to PR comments

3 participants

@felickz