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
Migration to oauthlib community #511
Comments
All great ideas @JonathanHuot! And what do people think about:
|
@JonathanHuot I'm trying clear things a little bit here #512 |
Anything I can do just ask. |
chiming in :) while working on a PR I noticed there is no coding style around which makes things sometimes hard to follow. I'd like to work on that if you guys are ok. Maybe starting with a issue containing a proposal and after that being accepted, starting to 💅 the codebase? |
Hi @MattBlack85, it's a good idea, any work in this direction is welcomed ! :-) |
re: coding style. I've found in projects I've worked on that by using autopep8 and yapf, I can basically let tooling clean up the coding style so I don't have to worry about it (except in cases where the cleaned up version is far less useful than having it un-cleaned-up, usually to do with line-lengths that would be clearer and only exceed the length boundary by a character or two). I use elpy-mode in Emacs to make that easy, but I suspect it could easily be done in command-line, and CI, as well. Having an |
Just thinking we probably want to have a run merging the currently open PRs before passing the code through a formatter. I'm all for pep8/flake8/yapf. |
👍 |
👍 I'm a big fan of just keeping things PEP8. |
Hi everyone,
Since @idan accepted the oauthlib community migration, as a team, we should list what do we need to progress as a true community. I suggest to start with a small list, and please anyone, feel free to participate by adding any suggestions :-)
Define/Improve release process:
Wrong releases numbers: we have 2.0.3 on
github's releases
, 2.0.5 in__init__.py
and 2.0.6 onpypi
...something wrong for sureI'd recommand to continue to use Travis for publishing, however we could define version directly by using environment variable
TRAVIS_TAG
(see example at https://github.com/thomsonreuters/bottle-oauthlib/blob/master/setup.py instead of our current hard-coded value: https://github.com/oauthlib/oauthlib/blob/master/oauthlib/__init__.py ). Also, I've seen @ib-lundgren is the actual publisher of the pypi package, he looks inactive since years, but dunno if it's an issue.README needs to be updated with the updated badge for travis-ci.org repository URL (owner must enable it, AFAIK), and also add a badge for the documentation build, cuz it's failing since long time: https://readthedocs.org/projects/oauthlib/builds/6483131/
DONE IN: Add shields for Python versions, license and RTD #520
No objection to continue using github issues and Google+, however it seems a bit outdated. Also, the #oauthlib IRC channel is empty,
wondering if we could create a Slack room if interested?CONCLUSION: gitter came to the rescue. Feel free to join at https://gitter.im/oauthlib/Lobby
Future, roadmap:
Also, a quick round table could be great. I start the introductions, I'm currently working on a OAuth2.0 RequestValidator implementation with bottle, and I've never worked with OAuth1.0, nor Django nor Pyramid nor Flask. However, I'm trying to have a good knowledge around the RFC involved here (oauth2, introspect, revocation, jwt...). I have not started yet the OpenID integration, but it will come soon.
The text was updated successfully, but these errors were encountered: