-
Notifications
You must be signed in to change notification settings - Fork 393
added tornadoAdapter. #144
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
base: master
Are you sure you want to change the base?
Conversation
|
After several years of inactivity, authomatic is now under community management, and we have just released a new stable version (1.0.0). We are now reviewing all issues and PRs and hoping to begin work to solve as many of these as possible. We are keen to find out which issues still apply, and which PRs are still required/are likely to merge cleanly into the current code. We are aiming to review them all, but any help with prioritisation would be very useful! If you are still interested in having this issue/PR resolved, or are able to help us work on it, please reply to this message. That way we know which issues are most important to the community. |
|
Ok, I am interested in resolving this issue/PR. Pls let me know if there is any requirement. |
|
Hi - that's great! The first thing to do is merge After that we need to think about how to add Tornado to the Let us know if you need any help with this! |
|
OK. So, let me check and resolve the conflicts.
|
Fix iteritems() -> items() Add missing re module import Handle empty argument lists to prevent IndexError Add unit tests covering functionality, and interface compliance Create functional test example following existing patterns Update tox configuration to include Tornado testing Disclaimer: This commit is created with LLM, also personall tested.
Added TornadoAdapter.