andreasvc / pyre2

Python wrapper for RE2
BSD 3-Clause "New" or "Revised" License
99 stars 33 forks source link

CI, python, and cmake cleanup #47

Closed sarnold closed 2 months ago

sarnold commented 2 months ago

I started this earlier in the year but then I got busy and had to add more fixes on top of fixes, but it's clean again. So

This does address C++ std warnings in issue #46 as well as #48, #44, #43, and stale/other version issues. You should probably validate the fix for #48 just to make sure

sarnold commented 2 months ago

So this is telling us there is still some Py2 cruft somewhere, possibly just a n import, or else it would not be looking for 'six'.