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

Add linting #1228

Open
4 tasks
Tracked by #1198
guarin opened this issue May 10, 2023 · 3 comments
Open
4 tasks
Tracked by #1198

Add linting #1228

guarin opened this issue May 10, 2023 · 3 comments
Labels
package

Comments

@guarin
Copy link
Contributor

guarin commented May 10, 2023

Description

@guarin finish writing issue and add subissues

Tasks

  • Lint the codebase and fix bad mistakes (unused variables, unused imports, …)
  • Add pylint to dev dependencies
  • Do this on a per subpackage basis
  • Check how much effort it would be to enforce linting
@guarin guarin mentioned this issue May 10, 2023
Open
18 tasks
@guarin guarin added this to Cleanup Aug 25, 2023
@guarin guarin added the package label Aug 16, 2024
@SSROCKS30
Copy link

Hey @guarin, I have been going to this repository and I am willing to solve this issue. Please can you assign me lightly\data package.

@SauravMaheshkar
Copy link
Collaborator

Related #1564

@guarin
Copy link
Contributor Author

guarin commented Oct 21, 2024 •
edited
Loading

Let's get ruff (#1564 ) merged and then we can start fixing linting errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
package
Projects
Cleanup
Status: No status
Milestone
No milestone
Development

No branches or pull requests
3 participants
@guarin @SauravMaheshkar @SSROCKS30