Skip to content

feat: Add support for testing local action against self-hosted GHE Servers #177

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ChrisFichman
Copy link

@ChrisFichman ChrisFichman commented Apr 2, 2025

Issue #, if available:
N/A

Description of changes:

This removes the restriction that the remote origin contains "github.com", so that users of self-hosted GitHub Enterprise servers can use the npm action.

Also improves the error message to be more specific.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Check any applicable:

  • Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

The original code has github.com hardcoded. This does not work for github enterprise servers where the hostname is different.

Also had to fix the owner/repo extraction to account for extra slashes in a remote host with weird pathing, and to check for the ":" which is required in an SSH url.
@ChrisFichman ChrisFichman changed the title Add support to running local action against self-hosted GitHub remotes Add support for running local action against self-hosted GitHub remotes Apr 2, 2025
@ChrisFichman ChrisFichman changed the title Add support for running local action against self-hosted GitHub remotes feat: Add support for running local action against self-hosted GHE Servers Apr 29, 2025
@ChrisFichman ChrisFichman changed the title feat: Add support for running local action against self-hosted GHE Servers feat: Add support for testing local action against self-hosted GHE Servers Apr 29, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant