-
Notifications
You must be signed in to change notification settings - Fork 109
Add session id to Session cmds, when appropriate #65
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
Conversation
Let's try to take the approach where we smooth out the |
541ad0c
to
22f7ea8
Compare
So I rebased my code to use |
Hmm looks like tests broke due to Py3. I'm still using Py2 as my main environment, I will take a look! |
Codecov Report
@@ Coverage Diff @@
## master #65 +/- ##
==========================================
+ Coverage 79.97% 80.07% +0.09%
==========================================
Files 8 8
Lines 844 848 +4
==========================================
+ Hits 675 679 +4
Misses 169 169
Continue to review full report at Codecov.
|
@askedrelic nice job! |
Hooray, thanks for the merge @tony 🎊 |
released in v0.7.4 |
Starting on fixing #64.