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

Linting setup #1

Merged
merged 3 commits into from
Oct 15, 2023
Merged

Linting setup #1

merged 3 commits into from
Oct 15, 2023

Conversation

Somtom
Copy link
Collaborator

@Somtom Somtom commented Oct 15, 2023

Problem

We are missing proper code formatting and contribution guidelines

What changed

  • Added contribution guidelines to the README
  • Added pre-commit configuration
  • Ran black, isort and flake 8 on the existing files, and applied changes

@Somtom Somtom self-assigned this Oct 15, 2023
@Somtom Somtom force-pushed the linting-setup branch 4 times, most recently from 229b150 to 8304a73 Compare October 15, 2023 12:59
@Somtom Somtom merged commit 3fb7944 into main Oct 15, 2023
2 checks passed
@Somtom Somtom deleted the linting-setup branch October 15, 2023 13:07
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.

1 participant