Add pylint, black & sort imports - Githubissues

digitalfabrik / lunes-cms

The back end for the Lunes vocabulary trainer. Front end: https://github.com/digitalfabrik/lunes-app.

https://lunes.app

Apache License 2.0

20 stars 8 forks source link

Add pylint, black & sort imports #498

Closed JoeyStk closed 1 year ago

JoeyStk commented 1 year ago

Short description

This PR adds a script the script file for pylint & for black. Also this PR sorts the order of import following the recommended order of import.

Proposed changes

Add pylint script
Add pylint to CI/CD pipeline
Resolve some of the pylint issues
Add black script
Sort order of import

Side effects

This PR touches a lot of files, including migration files. However to my knowledge none of the changes should touch any functionality and solely the code style
Since this PR by itself was already huge, I'm going to open a new issue to improve pylint further in a second step

Resolved issues

Fixes: #263

MizukiTemma commented 1 year ago

@timobrembeck Thank you for reveiws 🙂 the suggstions are applied.