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

ADD Black and isort #292

Merged
merged 4 commits into from
Nov 7, 2021
Merged

ADD Black and isort #292

merged 4 commits into from
Nov 7, 2021

Conversation

PIG208
Copy link
Contributor

@PIG208 PIG208 commented Nov 5, 2021

This extracts some dependencies-related changes from #283, configures black and isort, reformats files under /api, and adds lint checks to GitHub Actions.

This separates the python requirements for development from
production using .in files.

We can now use `make requirements` to generate the .txt
requirements from the .in files. This prepares for the
addition of mypy to the project.
Line length is set to 120.
@AlpacaMax
Copy link
Collaborator

Can you also add the instructions of using these linters into CONTRIBUTING.md?

@wabscale
Copy link
Collaborator

wabscale commented Nov 7, 2021

Thanks for the linting stuff. The one thing I would like you to remove is the linting check in the github action. I dont think we should consider tests as failed just because there is a linting issue.

@PIG208
Copy link
Contributor Author

PIG208 commented Nov 7, 2021

Thanks for the linting stuff. The one thing I would like you to remove is the linting check in the github action. I dont think we should consider tests as failed just because there is a linting issue.

Done

@wabscale wabscale merged commit 63cf74b into AnubisLMS:master Nov 7, 2021
PIG208 added a commit to PIG208/Anubis that referenced this pull request Nov 7, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Nov 7, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Nov 7, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Nov 23, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Nov 23, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Nov 23, 2021
isort was accidentally removed in AnubisLMS#292
@PIG208 PIG208 deleted the pyproj branch November 24, 2021 01:32
PIG208 added a commit to PIG208/Anubis that referenced this pull request Nov 27, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Dec 13, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Dec 13, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Dec 18, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Dec 18, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Dec 18, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Dec 25, 2021
isort was accidentally removed in AnubisLMS#292
PIG208 added a commit to PIG208/Anubis that referenced this pull request Dec 25, 2021
isort was accidentally removed in AnubisLMS#292
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.

None yet

3 participants