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

Automatic formatting via black + CI support #176

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

volkm
Copy link
Contributor

@volkm volkm commented Jul 19, 2024

  • Use black for automatic Python code formatting. After installing the black package, code can be formatted using black .
  • CI support for checking and applying code formatting, similar to the Storm and pycarl workflows.

@sjunges
Copy link
Contributor

sjunges commented Jul 28, 2024

Great. Does this format .rst, .ipynb and .py files?

@volkm
Copy link
Contributor Author

volkm commented Aug 7, 2024

Now it formats .py and .ipynb files ;)
.rst files are not formatted but they should also not contain any Python code anymore, because the documentation is in the notebooks.

@volkm
Copy link
Contributor Author

volkm commented Nov 18, 2024

@sjunges can I merge it?

@volkm
Copy link
Contributor Author

volkm commented Nov 19, 2024

Merging after positive feedback by Sebastian.

@volkm volkm merged commit 22f94d2 into moves-rwth:master Nov 19, 2024
9 of 10 checks passed
@volkm volkm deleted the autoformat branch November 19, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants