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 into
github:masterfrom
theoretick:master
Closed

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

Conversation

@theoretick

Copy link
Copy Markdown

No description provided.

@pengwynn

Copy link
Copy Markdown
Contributor

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

@owenthereal

Copy link
Copy Markdown
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
Copy Markdown
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

jdennes commented Jan 8, 2014

Copy link
Copy Markdown
Member

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
Copy Markdown
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

jdennes commented Jan 8, 2014

Copy link
Copy Markdown
Member

@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