Skip to content

ci(notifications): post PR comment with test result#74

Open
peter604 wants to merge 1 commit intoFullStackWithLawrence:mainfrom
peter604:main
Open

ci(notifications): post PR comment with test result#74
peter604 wants to merge 1 commit intoFullStackWithLawrence:mainfrom
peter604:main

Conversation

@peter604
Copy link
Copy Markdown

@peter604 peter604 commented May 5, 2026

Replace placeholder echo stubs with actions/github-script@v7. Calls GitHub REST API to comment on the originating PR with pass/fail status and a direct link to the run for triage.

Guard added so job only runs on pull_request events — issue_number is undefined on push triggers.

Pull Request Template

Type of Change

  • New feature
  • Bug fix
  • Documentation
  • Refactor
  • Chore

Resolves

  • Fixes #[Add issue number here.]

Changes

Describe what this Pull Request does

ci(notifications): post PR comment with test result
Replace placeholder echo stubs with actions/github-script@v7.
Calls GitHub REST API to comment on the originating PR with pass/fail
status and a direct link to the run for triage.

Guard added so job only runs on pull_request events — issue_number
is undefined on push triggers.

Testing

Describe the testing that has been done or needs to be done

Screenshots

Add any relevant screenshots

Dependencies

List dependencies

Breaking Changes

Describe any breaking changes

Replace placeholder echo stubs with actions/github-script@v7.
Calls GitHub REST API to comment on the originating PR with pass/fail
status and a direct link to the run for triage.

Guard added so job only runs on pull_request events — issue_number
is undefined on push triggers.
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.

1 participant