eclipse-qrisp / Qrisp

Qrisp - a framework for high-level programming of Quantum computers
https://www.qrisp.eu/
Eclipse Public License 2.0
93 stars 26 forks source link

Unify formatting by reinitiating Black #85

Open manzanillo opened 1 month ago

manzanillo commented 1 month ago

When working with the code I experienced a lot of different conventions being used when it comes to string indications ("" vs ''), spaces (e.g. def func1(param = "abc") vs def func1(param="abc")). In order to avoid to create too much noise in PR for new contributors putting Black (and maybe isort) back into use could be very helpful.

positr0nium commented 1 month ago

@bastibock what do you think about this?

bastibock commented 1 month ago

I agree! This definitely makes sense. We tried integrating it before but had some issues, especially when integrating into the GitLab CI/CD pipeline. But since the repo is on GitHub now, we should integrate it here. I will start working on it.