Conversation
Changes so checks are done with `ruff check .` instead of `flake8`.
|
CI is going to start failing this PR because the |
|
Tagging @kemccusker (if you're available) and @JMGilbert on this for review because you'll likely see this in CI code quality checks in other PRs once it merges. So, heads up. If you want, I can fix the issues it's flagging here now. Or we can deal with this in a later PR. 👍 |
Codecov Report
@@ Coverage Diff @@
## main #107 +/- ##
==========================================
- Coverage 70.34% 70.33% -0.02%
==========================================
Files 17 17
Lines 1878 1847 -31
==========================================
- Hits 1321 1299 -22
+ Misses 557 548 -9
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Seems like most of the fixes should be pretty benign (just removing unnecessary imports), so as long as this code quality package does everything we want it to do I'm happy with this update. |
|
Thanks, @JMGilbert. Appreciate the input. I'll commit the |
Changes so code quality checks are done with
ruff check .instead offlake8.Close #106.