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

Make TARDIS PEP8 complaint #2

Open
NanoNish opened this issue Oct 22, 2022 · 1 comment
Open

Make TARDIS PEP8 complaint #2

NanoNish opened this issue Oct 22, 2022 · 1 comment
Labels
codestyle

Comments

@NanoNish
Copy link
Member

Currently, TARDISis not PEP8 complaint.

To check this, install flake8 and run flake8 tardis.

To achieve this we need to:

  • Configure Flake8 through configuration files.
  • Ensure that Flake8 and Black play nicely together
  • Select appropriate Flake8 plugins, such as pep8-naming (discuss)
  • Make necessary changes to TARDIS code
  • more?

The CI pipeline is ready to check PEP8, just requires removing the if: false from the flake8 job.
@NanoNish
Copy link
Member Author

For original discussion go here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
codestyle
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@NanoNish