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

Cleanup indentation and whitespaces, and sort imports as per PEP8 #68

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pantheraleo-7
Copy link

@pantheraleo-7 pantheraleo-7 commented Dec 13, 2024

  • The top-level indent in SimpleMFRC522 module was 2 spaces, while the inner levels were 4. This PR changes every indentation to the standard 4.
  • This PR also trims the redundant whitespaces from the codebase.
  • This PR corrects various linter errors
  • Lastly, this PR sort and group the imports according to PEP8 specifications.

@pantheraleo-7
Copy link
Author

There's only one type error that remains after this at Line 141 in MFRC522.py.

@pantheraleo-7
Copy link
Author

pantheraleo-7 commented Dec 14, 2024

\cc @death-droid

(best reviewed commit by commit)

@pantheraleo-7 pantheraleo-7 changed the title Cleanup the indentation in SimpleMFRC522 module and whitespace and sort imports as per PEP8 Cleanup indentation and whitespaces, and sort imports as per PEP8 Dec 14, 2024
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