Skip to content
This repository has been archived by the owner on Feb 7, 2025. It is now read-only.

Support GHES #7

Merged
merged 3 commits into from
Jan 10, 2025
Merged

Support GHES #7

merged 3 commits into from
Jan 10, 2025

Conversation

wechuli
Copy link
Contributor

@wechuli wechuli commented Feb 22, 2024

Closes #6

Currently, the action won't work on GHES because it does not account for the baseUrl property in the Octokit contructor. This change takes into account that it might be running on a GHES instance and reads the baseUrl property from the environment variable - GITHUB_API_URL available on each run.

Copy link
Contributor

@jhutchings1 jhutchings1 left a comment

Choose a reason for hiding this comment

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

Changes look good, thanks @wechuli !

@lukegeor
Copy link

lukegeor commented Mar 5, 2024

Thank you for the quick turnaround!

@giotab
Copy link

giotab commented Jan 10, 2025

Any plans on merging this?

@lseppala lseppala merged commit 6fe43ab into advanced-security:main Jan 10, 2025
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for GitHub Enterprise?
5 participants