Skip to content
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

Require that Python code conform to the Black formatter #2251

Open
anshumanmohan opened this issue Aug 6, 2024 · 1 comment
Open

Require that Python code conform to the Black formatter #2251

anshumanmohan opened this issue Aug 6, 2024 · 1 comment
Assignees
Labels
C: calyx-py Items that have to do with the builder library

Comments

@anshumanmohan
Copy link
Contributor

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?

@anshumanmohan anshumanmohan added the C: calyx-py Items that have to do with the builder library label Aug 6, 2024
@polybeandip polybeandip self-assigned this Aug 6, 2024
@polybeandip
Copy link
Collaborator

Happy to set up the github action for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: calyx-py Items that have to do with the builder library
Projects
None yet
Development

No branches or pull requests

2 participants