Skip to content

ci(Release Notes): add missing environment variables#2169

Merged
AprilSylph merged 1 commit into
masterfrom
aprilsylph/release-notes-env-complete
Mar 26, 2026
Merged

ci(Release Notes): add missing environment variables#2169
AprilSylph merged 1 commit into
masterfrom
aprilsylph/release-notes-env-complete

Conversation

@AprilSylph

@AprilSylph AprilSylph commented Mar 25, 2026

Copy link
Copy Markdown
Owner

Description

Adds GITHUB_API_URL and GITHUB_REPOSITORY environment variables to the Release Notes workflow.

Adding the former is more of a completeness thing (it's available on the github context, so why not use it) while the latter should enable this workflow to work properly on forks, instead of always querying AprilSylph/XKit-Rewritten.

Testing steps

See https://github.com/AprilSylph/XKit-Rewritten/actions/runs/23543601713 to verify that the workflow is not broken in the source repository by this change.

To validate that this has the intended effect on forks, you will need to:

  1. Copy this branch to your fork
  2. Drop one or more src/-affecting commits from your fork's default[?] branch (or just don't update it, if it's behind upstream), to make it differ
  3. Run the modified workflow in your fork
    • Expected result: The commits printed match those present on your fork's default[?] branch

@AprilSylph AprilSylph marked this pull request as ready for review March 25, 2026 13:37
@AprilSylph AprilSylph merged commit 29ef399 into master Mar 26, 2026
6 checks passed
@AprilSylph AprilSylph deleted the aprilsylph/release-notes-env-complete branch March 26, 2026 09:39
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