-
Notifications
You must be signed in to change notification settings - Fork 79
Stop exposing All
and One
methods
#119
Comments
I dig this approach, @calavera.
Maybe
I think there's some value in keeping these public for extensibility.
Looking forward to seeing your approach. |
👍 To @calavera's proposal of adding "friendly" functions that don't need the map, wether or not the current map-needy functions remain. |
@pengwynn, @calavera: Any interest in me taking one of the API endpoint groups and sending a PR with what this might look like, so you can make a call on whether to go down this path? I'm happy to help with the work, but don't want to do too much of it before a decision is made. If so, what part of the API would you like me to prototype this with? |
@half-ogre Absolutely!
I don't think it matters much, but Issues might be a good place to start. |
Please, feel free to carry this code in any way. Unfortunately, I'm very busy with other "small" open source project to polish this how I'd like it. |
So, coincidentally, I'm using go-octokit for a project I'm working on and just happen to need the "team" APIs... so I thought I'd contribute. Once of the things I noticed as I was working on a non-One/All implementation was that I was re-writing a lot of boilerplate code (basically, the stuff that's already in |
Yes I think this should be put into the API. Pin the issue? |
Users should not need to know how go-octokit generates hypermedia urls internally. The fact that people must provide maps with aleatory keys to those methods to call the api is a bad smell. Octokit should provide high level functions to perform every operation. Something like:
And so on.
I'll open a PR soon with a few of those methods, but providing all of them can be a community effort.
Feel free to claim resources that you want to modify yourself commenting in this issue.
TODO:
The text was updated successfully, but these errors were encountered: