add check for E221,E222,E227,E228 to the linter and extend it to dev/

[fchapoton]

this is

• extending the linter checks to the dev/ folder
• adding checks for E221, E222, E227 and E228 (all about spaces around operators)

[NathanDunfield]

Looks like these errors are from building the the binary extension FXrays for Python 3.12, not your changes. But we should get everything working with Python 3.12 first to be sure, see #112.

[culler]

I think I have fixed FXRays. With Cython 3.0 the except clause apparently needs to be included when declaring a function pointer. - Marc

…

On Sun, Aug 13, 2023 at 1:52 PM Nathan Dunfield ***@***.***> wrote: Looks like these errors are from building the the binary extension FXrays <https://github.com/3-manifolds/FXrays> for Python 3.12, not your changes. But we should get everything working with Python 3.12 first to be sure, see #112 <3-manifolds/SnapPy#112>. — Reply to this email directly, view it on GitHub <#53 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJ6CP3KFZ4DUYDQEHAWUV3XVEO5VANCNFSM6AAAAAA3OQRPVY> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[fchapoton]

so it this now ready to merge ?

[NathanDunfield]

so it this now ready to merge ?

Probably. I think we've updated all the dependencies of spherogram to support Python 3.12, so try running the CI again. If t's all green, I'll merge.