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

Implement missing endpoints and small fixes #334

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ianmaciel
Copy link

Hello,

I had to update some parts of the code to use on my project.
These are the modifications included:

Would you like to include theses changes on your project?

Best regards,
Ian Maciel

Implement the service to create a new repo using a template according
https://docs.github.com/en/rest/repos/repos#create-a-repository-using-a-template
specs (endpoint: /repos/{template_owner}/{template_repo}/generate)
Current Github docs specify most of the parameters as optional. This
commit update the existing editRepository method to these specs.

https://docs.github.com/en/rest/repos/repos#update-a-repository
Update the existing method addCollaborator to meet the current docs.
https://docs.github.com/en/rest/collaborators/collaborators#add-a-repository-collaborator

Instead of NO_CONTENT, the expected status code is CREATED.
@robrbecker
Copy link
Member

Thanks for putting up a PR. I'll take a look.

@ianmaciel
Copy link
Author

I'm sorry about the delay. I completely forgot this PR.
I have just updated with the suggested changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants