[MNT] Remove PyYAML and Cython CI workaround - #140
Conversation
d6ce99c to
e13cd54
Compare
fkiraly
left a comment
There was a problem hiding this comment.
Can you kindly explain why we can simply remove this?
Also, there are merge conflicts with main.
The patch is just probing if we could remove this snippet per CI outcome, and to gather responses from the community that say we shouldn't. Do you have any concerns about removing Cython because of performance regressions?
The PR is stacked on another one and got out of sync. Please don't worry: I don't always keep all things in sync to save CI minutes, but will resolve conflicts before merging, when applicable. |
No, I simply do not have enough information of what the snippet does and what happens if we remove it, that´s all. I was asking for more info. |
|
Let's toggle the patch into draft mode again and wait for @WilliamJudge94 to possibly come up with answers to those details later? |
About
References