Skip to content

Add pull-request-id output#1150

Merged
int128 merged 1 commit intomainfrom
int128/Add-pull-request-id-output
Mar 17, 2026
Merged

Add pull-request-id output#1150
int128 merged 1 commit intomainfrom
int128/Add-pull-request-id-output

Conversation

@int128
Copy link
Copy Markdown
Owner

@int128 int128 commented Mar 17, 2026

No description provided.

@int128 int128 marked this pull request as ready for review March 17, 2026 11:53
@int128 int128 requested a review from Copilot March 17, 2026 11:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new GitHub Action output (pull-request-id) to expose the created pull request’s GraphQL node ID (node_id) to downstream workflow steps.

Changes:

  • Add pullRequestId to the action’s internal Outputs type and set it from pull.node_id when a PR is created.
  • Emit the new pull-request-id output from the action entrypoint and document it in action.yaml and README.md.
  • Update the existing handleOtherEvent test to mock and assert the new output.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/other_event.test.ts Extends PR creation mock/expectations to include node_idpullRequestId.
src/run.ts Adds pullRequestId?: string to the exported Outputs type.
src/other_event.ts Captures pull.node_id and returns it as pullRequestId; updates local PullRequest type and dry-run stub.
src/main.ts Sets the new pull-request-id action output when present.
action.yaml Documents the new pull-request-id output.
README.md Documents pull-request-id in the Outputs table.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@int128 int128 merged commit 4916d7b into main Mar 17, 2026
7 checks passed
@int128 int128 deleted the int128/Add-pull-request-id-output branch March 17, 2026 23:44
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