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 tests for 3.10 #817
Added tests for 3.10 #817
Conversation
Added tests CI check on python 3.10 for pygraphviz install Trying to fix R issus
Added tests CI check on python 3.10 for pygraphviz install Trying to fix R issus
is this ready for review? I see one windows test failing but that one fails intermittently (haven't been able to fix it) |
I see in windows 3.10 there are a few that's failing and probably need the skip flag. |
ah yeah. I see them. it's because they need pygraphviz. I think let's only skip them when running on Python 3.10 and windows (so both conditions) |
pygraphviz had issues with 3.10 so it's disabled on linux and macos as
well, so the current condition is <3.10, I've added those skips lets see if
it's passing now.
…On Tue, May 31, 2022 at 8:45 AM Eduardo Blancas ***@***.***> wrote:
ah yeah. I see them. it's because they need pygraphviz. I think let's only
skip them when running on Python 3.10 and windows (so both conditions)
—
Reply to this email directly, view it on GitHub
<#817 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACYPJOMR2FRZIQBENMOMC7DVMYCXXANCNFSM5XEFXXVQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
got it! |
the CI is failing but I think it's because of the intermittent issue. lmk when it's ready for review |
I think it's ready there are some windows 3.10 tests that are failing even with the skip, not sure if thats the intermittent ones. Also you might see more files since I rebased on the master |
Why do we need an extra branch here? |
i wanted to trigger an extra run to see the CI, will merge shortly |
No description provided.