[FEAT] Enable Ruff

[samster25]

• Enable ruff to speed up linting and increase the number of rules we're using

[codecov]

Codecov Report

Attention: Patch coverage is 56.52174% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 85.09%. Comparing base (e2c6fec) to head (04915bb).
Report is 5 commits behind head on main.

❗ Current head 04915bb differs from pull request most recent head 649d736. Consider uploading reports for the commit 649d736 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2121      +/-   ##
==========================================
+ Coverage   85.05%   85.09%   +0.04%     
==========================================
  Files          68       68              
  Lines        7332     7326       -6     
==========================================
- Hits         6236     6234       -2     
+ Misses       1096     1092       -4     

Files	Coverage Δ
daft/__init__.py	24.24% <ø> (ø)
daft/delta_lake/delta_lake_scan.py	86.66% <100.00%> (ø)
daft/execution/execution_step.py	93.39% <100.00%> (-0.02%)	⬇️
daft/execution/physical_plan.py	94.86% <100.00%> (ø)
daft/execution/rust_physical_plan_shim.py	95.83% <100.00%> (ø)
daft/filesystem.py	60.89% <ø> (ø)
daft/hudi/hudi_scan.py	82.35% <ø> (ø)
daft/hudi/pyhudi/table.py	97.61% <ø> (ø)
daft/io/_delta_lake.py	89.36% <ø> (ø)
daft/logical/builder.py	91.52% <100.00%> (-0.08%)	⬇️
... and 11 more

[MeepoWin]

Amazing! I'm looking forward to how much this will reduce pre-commit wait times. :)

[samster25]

Amazing! I'm looking forward to how much this will reduce pre-commit wait times. :)

@MeepoWin me too! There's also so many Ruff rules to avoid footguns :)

[clarkzinzow]

Do we want to exclude vendored files, like the cloudpickle code? We'd ideally like to keep vendored files as close to their source as possible, in case we want to pull in upstream changes.

[clarkzinzow]

I know this isn't the time to litigate this, but at some point I'd like to discuss dropping this to 80 or Black's default of 88, since 120 often isn't great for reviewing PRs or editing horizontal code panes on a 14-16" laptop.

A discussion for the future! 😉

[samster25]

Yeah happy to discuss offline! Not super attached to this

[samster25]

Do we want to exclude vendored files, like the cloudpickle code? We'd ideally like to keep vendored files as close to their source as possible, in case we want to pull in upstream changes.

Done! now ignore the cloud pickle files.