-
Notifications
You must be signed in to change notification settings - Fork 294
Document our public API #307
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
Comments
Is this being worked on at all? |
@spacekitteh Pretty much no; I basically got burnt out from a bunch of OSS stuff I was doing in my spare time, this included, so pretty much nothing has happened since the last release. |
Sorry to hear :( |
FWIW, I have two separate transatlantic trips next month, so I'm hopeful of getting something done then because flights to the West Coast are just Too Damned Long to not run out of things to do. Half the problem with documenting the API is the fact that some of the arcane parts I don't really know what the API even is. There's basically two halves: the half that almost anyone could write (realistically, this extends little beyond what's included in If anyone wants to help with the former, absolutely do! I can't promise I'll be great at dealing with PRs, though, for the reasons above… :\ Even if I don't get back to you in a timely manner, there's every chance your work will end up getting included when I do start working on it again. |
I think the punch list is something like this:
I'll go through these one at a time, do a first pass, and set up a PR. |
The work here finished except for any post-review work to do. Just waiting on @gsnedders to review PR #386. |
For semver to make any sense, we really need to actually define what our public API is. We probably want to do this both through documentation and through the de-facto Python standard of underscore-prefixing (though that will likely break stuff again, le sight, but if people are using the private API they'll likely complain when we do break it in some 1.x release).
The text was updated successfully, but these errors were encountered: