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

pre-commit.ci #66

Merged
merged 4 commits into from
Sep 1, 2022
Merged

pre-commit.ci #66

merged 4 commits into from
Sep 1, 2022

Conversation

Synss
Copy link
Owner

@Synss Synss commented Sep 1, 2022

The PR fulfills these requirements

  • This change is a single feature in a single commit.
  • The commit message follows our commit guidelines.
  • The code formatting follows our coding guidelines.
  • Tests have been added for the changes.
  • Docs have been added / updated (docstrings and README).
  • A one-liner has been added to the ChangeLog (for new features).

More details in CONTRIBUTING.

I am submitting a …

  • Bug fix
  • New feature
  • Tests update (new test vector, etc.)
  • Docs update
  • Other (please complete)

Description

Other information

black is our SSOT regarding formatting issues.
Also remove pylint (currently unused) and add pre-commit
to requirements/tests.in
@coveralls
Copy link

coveralls commented Sep 1, 2022

Coverage Status

Coverage remained the same at 90.945% when pulling 32161dd on dev into 408108e on master.

That's where the cool kids are 😎
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.

2 participants