-
Notifications
You must be signed in to change notification settings - Fork 6k
added python version note to the luci script #27929
Conversation
|
This pull request is not suitable for automatic merging in its current state.
|
|
Can we just try to patch the script to not fail on Python 3.9? If that is not trivial, can we document the issue in a bug for followup? |
|
From triage, another data point from Zach: He is running 3.9.2 and that seems to be fine. We should definitely dig a bit deeper to figure out why newer versions might be no good. |
|
@zanderso @chinmaygarde Okay, I removed the note about 3.9. We still need a note about 3.7. We'll consider anything that doesn't work beyond 3.7 a bug. I'll see if I can reproduce the 3.9 bug. The problem would only arise for me if a rebuild was actually triggered fwiw, it's a runtime error. |
|
sgtm. I checked my python version and it was 3.8.2. I'll update it and be on the lookout for this issue as well. |
b09578e to
7e2c1c7
Compare
Pre-launch Checklist
writing and running engine tests.
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.