Skip to content
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

added tornadoAdapter. #144

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

laminko
Copy link

@laminko laminko commented Apr 10, 2016

Added TornadoAdapter.

@mrichar1
Copy link
Member

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.

@laminko
Copy link
Author

laminko commented Dec 9, 2019

Ok, I am interested in resolving this issue/PR. Pls let me know if there is any requirement.

@mrichar1
Copy link
Member

Hi - that's great!

The first thing to do is merge master into your branch and resolve the conflicts. This should then allow a basic set of tests to run, which will verify that there have been no major breakages in your PR.

After that we need to think about how to add Tornado to the liveandletdie project so that it can be called in testing to ensure it is working properly.

Let us know if you need any help with this!

@laminko
Copy link
Author

laminko commented Dec 10, 2019

OK. So, let me check and resolve the conflicts.

  • add some unit-tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants