feat: add support for github enterprise server - Githubissues

gruntwork-io / git-xargs

git-xargs is a command-line tool (CLI) for making updates across multiple Github repositories with a single command.

https://blog.gruntwork.io/introducing-git-xargs-an-open-source-tool-to-update-multiple-github-repos-753f9f3675ec

Apache License 2.0

921 stars 63 forks source link

feat: add support for github enterprise server #132

Closed kennyg closed 7 months ago

kennyg commented 11 months ago

Description

This PR adds support for Github Enterprise. I used this PR as a guide, and added a small test.

TODOs

Read the Gruntwork contribution guidelines.

[x] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added support for Github Enterprise servers.

kennyg commented 10 months ago

@brikis98 can you help with an approval for this PR?

andyfeller commented 8 months ago

@brikis98 @zackproser : is there someone else at Gruntworks who might be capable of reviewing and approving this PR? 🙏

zackproser commented 7 months ago

@andyfeller you could try @josh-padnick or @hongil0316 ?

josh-padnick commented 7 months ago

@gitsstewart Could you identify an SME to help review this PR?

gitsstewart commented 7 months ago

I will ask the team if anyone is available quickly for a review

james03160927 commented 7 months ago

Will check when it complets - https://app.circleci.com/pipelines/github/gruntwork-io/git-xargs/332/workflows/aafce9b1-fd99-4708-9b0a-8ab1ad5a374b

james03160927 commented 7 months ago

Confirmed that all the tests passed. Will merge.

thirstydeveloper commented 4 months ago

This will be very helpful to my team!

@hongil0316 - could you release a new version with this feature?

josh-padnick commented 4 months ago

@hongil0316 I'll go ahead and issue the release since the only thing it includes is this change.

josh-padnick commented 4 months ago

Released in v0.1.11

james03160927 commented 4 months ago

Oh sorry about that @thirstydeveloper, I must have missed this. Thanks @josh-padnick for taking it! :)