-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
GraphQL API for Enterprise GitHub #2870
Comments
True, should be configurable like |
@EnricoMi I have been using PyGithub in our project for over a year. If no one is working on this, can I open a PR and try to resolve this issue ? |
@shicholas are you planning to create a PR? |
@EnricoMi I haven't had the time yet, @samad-yar-khan it's all you, I'll probably need another week then I can look into this. |
@samad-yar-khan go for it, @shicholas happy to get your review when you have time. |
@EnricoMi Thanks, I will look into into it and try solving this issue. |
@samad-yar-khan @EnricoMi threw up a PR, thanks! |
@shicholas @EnricoMi I couldn't open a PR within time as I was still navigating the code, had some other time constraints and saw a PR already opened for the issue. Thanks for the PRs, went through them to learn a bit about the codebase. |
thanks so much @EnricoMi! |
When using the GraphQL API such as for features like
enable_auto_merge
, a user should be allowed to specify the URL. AFAICT it's hardcoded https://github.com/PyGithub/PyGithub/blob/603896f44541740d72aa441ed60c698760a1393e/github/Requester.py#L543C81-L543C81The text was updated successfully, but these errors were encountered: