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.

Sign up for GitHub

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

Jump to bottom

Before main merge we should reformat/lint the whole codebase #16

Open
schettn opened this issue Nov 24, 2020 · 0 comments
Open

Before main merge we should reformat/lint the whole codebase #16

schettn opened this issue Nov 24, 2020 · 0 comments
Assignees
@kleberbaum @schettn @neuwirte
Labels
dif/trivial Can be confidently tackled by newcomers effort/days Estimated to take multiple days, but less than a week kind/maintenance Work required to avoid breaking changes or harm to project's status quo
Milestone
v1.0

Comments

@schettn
Copy link
Contributor

schettn commented Nov 24, 2020 •
edited
Loading

Describe the feature or change you'd like
By running the commands below: imports are sorted, unused imports removed and
everything is formatted by black.

isort -rc .
autoflake -r --in-place --remove-all-unused-imports --remove-unused-variables .
black .

Additional context

Ref: https://github.com/psf/black

Install instructions:

pip3 install isort
pip3 install autoflake
pip3 install black
@schettn schettn added dif/trivial Can be confidently tackled by newcomers effort/days Estimated to take multiple days, but less than a week kind/maintenance Work required to avoid breaking changes or harm to project's status quo labels Nov 24, 2020
@schettn schettn changed the title Before master merge we should reformat/lint the whole codebase Before main merge we should reformat/lint the whole codebase Nov 24, 2020
@schettn schettn added this to the v1.0 milestone Dec 1, 2020
schettn added a commit that referenced this issue Jan 7, 2021
@schettn
Reformat track/user app codebase #16
50f96a5 
The track and user app have been formatted/linted due to
#16.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kleberbaum kleberbaum

@schettn schettn

@neuwirte neuwirte

Labels
dif/trivial Can be confidently tackled by newcomers effort/days Estimated to take multiple days, but less than a week kind/maintenance Work required to avoid breaking changes or harm to project's status quo
Projects
None yet
Milestone
v1.0
Development

No branches or pull requests
3 participants
@kleberbaum @schettn @neuwirte