qa: configure linting to fail on >80 columns wrapping
Link issues together to show that they're related or that one is blocking others. Learn more.
Activity
- Marin Hannache added QA label
added QA label
- Author Contributor
Django is switching to Black for code formatting. Maybe we should follow the same convention.
Edited by Marin Hannache - Contributor
Black does not support any form of xml output. However it does support a diff output ou stdout but as far I know gitlab can't understand this format. It will be difficult to have a seamless integration with gitlab... We can, however stick with pylint as a syntax checker.
But we can still incorporate black into our project. We can either add black as a pre-commit hook (https://github.com/python/black/blob/master/README.md#version-control-integration) or put a script to execute it. Projects that use black use both of these techniques.
Black recommend that we use 88 as a max length for columns. I suggest that we do the same.
Edited by Arthur Outhenin-Chalandre - Arthur Outhenin-Chalandre mentioned in merge request !20 (merged)
mentioned in merge request !20 (merged)
- Arthur Outhenin-Chalandre created merge request !20 (merged) to address this issue
created merge request !20 (merged) to address this issue
- Arthur Outhenin-Chalandre added pending review label
added pending review label
- Marin Hannache closed via merge request !20 (merged)
closed via merge request !20 (merged)
- Marin Hannache removed pending review label
removed pending review label