-
-
Notifications
You must be signed in to change notification settings - Fork 141
Experiment2 #247
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
Experiment2 #247
Conversation
In this case, it failed because changing So the issue now is that if we do update |
That is also my conclusion |
One thing to try is to use |
I think I tested disabling the poetry.lock caching but adding my local poetry.lock: it still failed (but maybe I'm misremembering something) edit: tested it here dfba485 |
It's the other way that is not working. If the cache is invalid, then the CI is failing. |
In theory, my local poetry.lock should be the same as the cached poetry.lock. When disabled caching and adding my poetry.lock, it should have the same effect - but it would always be a cache hit. |
Trying to skip the cache on windows. |
I tried that last night. It doesn't solve the problem. The issue here is that if there is no cache, it will fail. If we do get the cache, then it succeeds. So we need to know that when we have a new version of |
Is there a way to download the cached poetry.lock so that we can compare it to a newly generated one? |
If you know the location could just cat them.
…On Thu, Sep 1, 2022, 20:51 Torsten Wörtwein ***@***.***> wrote:
Is there a way to download the cached poetry.lock so that we can compare
it to a newly generated one?
—
Reply to this email directly, view it on GitHub
<#247 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKTSRMPCP7SYX44CVF25KDV4ECLFANCNFSM6AAAAAAQCLUQGY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Would just need to do edit: well if the cache was still there - which is not the case on windows |
I don't think that it matters. As soon as we update Also, I think I read on the action for installing poetry that they discovered that caching doesn't work on Windows anyway |
Should carefully check if any of the deps have changed from when it last
worked. Could be something else, like virtualenv.
…On Thu, Sep 1, 2022, 20:57 Irv Lustig ***@***.***> wrote:
Is there a way to download the cached poetry.lock so that we can compare
it to a newly generated one?
I don't think that it matters. As soon as we update pyproject.toml, there
will be no valid cached file, and we'll run into the current problem,
because the CI will create a new poetry.lock file.
—
Reply to this email directly, view it on GitHub
<#247 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABKTSRLO73F754RLLUBVRZ3V4EDBPANCNFSM6AAAAAAQCLUQGY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Pinning poethepoet to 0.16.0 (vs 0.16.1) fixes the problem. |
Oooh. That's good news. I will test in what I'm trying just to be sure. |
It might be worth pinning all versions in pyproject.toml (and also the github action versions to their exact version). We will need to update them every now and then but we might be spared a few days of debugging :) |
Closing in favor of #249 |
Created nat-n/poethepoet#88 |
assert_type()
to assert the type of any return value