Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix warnings #299

Merged
merged 2 commits into from
Jun 20, 2024
Merged

Fix warnings #299

merged 2 commits into from
Jun 20, 2024

Conversation

naik-aakash
Copy link
Collaborator

Closes #287

Changes

  1. Enable flake8 in ci
  2. add stack level to warnings

@naik-aakash
Copy link
Collaborator Author

Hi @JaGeo ,

this PR could be reviewed and merged

Copy link

codecov bot commented Jun 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.36%. Comparing base (fdfbf90) to head (60cc9f7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #299   +/-   ##
=======================================
  Coverage   97.36%   97.36%           
=======================================
  Files          10       10           
  Lines        2660     2660           
=======================================
  Hits         2590     2590           
  Misses         70       70           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JaGeo JaGeo merged commit 55d8d2e into JaGeo:main Jun 20, 2024
32 checks passed
@naik-aakash naik-aakash deleted the fix_warnings branch October 18, 2024 07:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix warnings for flake8 pre-commit CI
2 participants