Skip to content
This repository was archived by the owner on Nov 1, 2017. It is now read-only.

Add go-octokit to official library list#398

Closed
theoretick wants to merge 1 commit intogithub:masterfrom
theoretick:master
Closed

Add go-octokit to official library list#398
theoretick wants to merge 1 commit intogithub:masterfrom
theoretick:master

Conversation

@theoretick
Copy link

No description provided.

@pengwynn
Copy link
Contributor

@jingweno How far are we from a one-point-oh?

@owenthereal
Copy link
Contributor

@pengwynn I'm using go-octokit in gh and the current design has been stable for a while, although not all API endpoints have been implemented. The current API, IMHO, is quite enjoyable to use.

I chatted with @technoweenie earlier though and we agree we should explore building a smarter hypermedia client with URL caching etc. That says potentially we'll break the current design in go-octokit. But we didn't come to a conclusion on how to move forward.

@theoretick
Copy link
Author

Ah, apologies if go-octokit isn't ready for getting listed, thought its absence was just an oversight. But cool, it its stable enough glad to prompt the discussion.

@jdennes
Copy link
Member

jdennes commented Jan 8, 2014

octokit.rb is actually the only official library which is currently >= 1.0.0. Both octokit.objc and octokit.net are < 1.0.0.

@technoweenie
Copy link
Contributor

@jdennes: Go libraries are more strict about maintaining backwards compatibility because the import system lacks any real versioning. There are ways around it of course, but users of your library may have problems if you introduce breaking changes to master.

So really, once we add it to the list and start encouraging people to use go-octokit, it kind of has to be 1.0.

@jdennes
Copy link
Member

jdennes commented Jan 8, 2014

@technoweenie: 🆒 — closing this for now then.

@jdennes jdennes closed this Jan 8, 2014
izuzak added a commit that referenced this pull request Dec 8, 2014
Add missing pagination headers to example responses
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants