search

calyxir / calyx

Intermediate Language (IL) for Hardware Accelerator Generators
https://calyxir.org
MIT License
473 stars 46 forks source link

Require that Python code conform to the Black formatter #2251

Open anshumanmohan opened 1 month ago

anshumanmohan commented 1 month ago

Let's expand the scope of mandatory Black formatting in CI. Currently only two directories are tested, but I think this can safely be expanded to include all of calyx-py for instance.

Right now if I run black calyx-py I get

am3327@havarti:/scratch/anshuman/calyx$ black calyx-py/
...
All done! ✨ 🍰 ✨
22 files reformatted, 32 files left unchanged.

@polybeandip interested? Since it bit you once before and you figured this out in OCaml-land recently?

polybeandip commented 1 month ago

Happy to set up the github action for this