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 auto formatter #325

Draft
wants to merge 49 commits into
base: develop
Choose a base branch
from
Draft

🎨 Add auto formatter #325

wants to merge 49 commits into from

Conversation

mayankpatibandla
Copy link
Member

@mayankpatibandla mayankpatibandla commented Feb 2, 2024

Summary:

Adds black formatter and isort as pre-commit hooks to automatically format changes. These hooks have been run on every python file in the repo.

Motivation:

These auto formatters will lead to a more consistent coding style.

References (optional):

#301

Test Plan:

  • Ensure all hooks run
  • Ensure GitHub actions pass
  • This PR only includes formatting changes so functionality should be unaffected

@mayankpatibandla mayankpatibandla self-assigned this Feb 2, 2024
@mayankpatibandla mayankpatibandla mentioned this pull request Mar 5, 2024
90 tasks
@mayankpatibandla mayankpatibandla changed the title ✨ Add auto formatter 🎨 Add auto formatter Mar 27, 2024
@mayankpatibandla mayankpatibandla changed the base branch from develop to pylint-ci March 28, 2024 00:04
@mayankpatibandla mayankpatibandla changed the base branch from pylint-ci to develop March 28, 2024 00:05
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