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
Version update - New branch #558
Conversation
is this ready for review? |
monkeypatch=monkeypatch, | ||
last_check='2022-01-20 10:51:41.082376') # version='0.14.8', | ||
|
||
# Test no warning when same version encountered |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove commented code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait, what is this testing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to test no echo was printed when it's the same version, for some reason the patch I tried on the ploomber.version didn't work, not sure why.
check errors, the build is failing |
tests pass. is this ready for review? |
Give me today to run some more tests on it :)
…On Mon, Feb 14, 2022 at 12:00 PM Eduardo Blancas ***@***.***> wrote:
tests pass. is this ready for review?
—
Reply to this email directly, view it on GitHub
<#558 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACYPJONO23NS3VLCEOZTIJLU3EYKZANCNFSM5NYETLSQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
sure! I quickly skimmed through the code: can you add a test for the timeout? I see the |
… on function runtime
@edublancas ready for review, I've decoupled the files and added the timeouts check |
No description provided.