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

Adds automated linting and linting workflow; standardized all code according to black style #2

Merged
merged 18 commits into from
Apr 30, 2024
Merged

Adds automated linting and linting workflow; standardized all code according to black style #2

merged 18 commits into from
Apr 30, 2024

Conversation

andreped
Copy link
Member

@andreped andreped commented Apr 29, 2024 •
edited
Loading

Most of the changes are due to the application of the new linting style.

Changes

  • Implement shell scripts for testing linting and automated refactoring
  • Implement workflow for running linting tests for any PR, commit, and on dispatch
  • Apply linting style to entire codebase (*.py files only, not Jupyter Notebooks)
  • Update README to include instructions on how to perform linting
  • Add requirements file with relevant packages

@SoroushOskouei SoroushOskouei merged commit 7d5ade4 into main Apr 30, 2024
2 checks passed
@andreped andreped deleted the linting branch April 30, 2024 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SoroushOskouei SoroushOskouei Awaiting requested review from SoroushOskouei

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andreped @SoroushOskouei