Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: add option to override base url for GHES #104

Merged
merged 1 commit into from
Dec 21, 2022

Conversation

koalaty-code
Copy link
Contributor

Adds an option for --octoherd-base-url to configure underlying Octokit client for GHES. The URL passed to --octoherd-base-url will be passed as baseUrl to the underlying Octokit client.

There isn't a current testing pattern in place for the existing options. Open to feedback/ideas on how to add tests for options if desired.

Copy link
Member

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

There isn't a current testing pattern in place for the existing options. Open to feedback/ideas on how to add tests for options if desired.

Open to ideas as well 🤣 If you tested it locally and it works, that's good enough for me

@travi
Copy link
Member

travi commented Dec 21, 2022

we've verified locally that the change works as intended. happy to discuss ideas for further automated testing as a follow up

@gr2m gr2m merged commit 5c62399 into octoherd:main Dec 21, 2022
@github-actions
Copy link

🎉 This PR is included in version 3.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@koalaty-code koalaty-code deleted the octoherd-base-url-option branch December 21, 2022 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants