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

Fix code formatting for all files in the repository once and for all #1682

Closed
jougs opened this issue Jul 6, 2020 · 0 comments · Fixed by #2105
Closed

Fix code formatting for all files in the repository once and for all #1682

jougs opened this issue Jul 6, 2020 · 0 comments · Fixed by #2105
Assignees
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.
Milestone

Comments

@jougs
Copy link
Contributor

jougs commented Jul 6, 2020

It seems that not all files in the repository currently conform to our static code style guidelines and people are sometimes hit by this when they edit a file, pertaining files are then checked and unrelated code locations evoke errors.

In order to ease future development, all files in the repository should again be checked for conformity (PEP8, clang-format) and committed.

@jougs jougs added S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) labels Jul 6, 2020
@heplesser heplesser added this to the NEST 3.2 milestone Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants