search

reata / sqllineage

SQL Lineage Analysis Tool powered by Python
MIT License
1.19k stars 215 forks source link

feat: respect sqlfluff configuration file #582

Closed reata closed 2 months ago

reata commented 4 months ago

Closes #548

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.50%. Comparing base (3574f2d) to head (950b510). Report is 3 commits behind head on master.

:exclamation: Current head 950b510 differs from pull request most recent head 54114c9. Consider uploading reports for the commit 54114c9 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #582 +/- ## ======================================= Coverage 99.50% 99.50% ======================================= Files 41 41 Lines 2219 2219 ======================================= Hits 2208 2208 Misses 11 11 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

reata commented 4 months ago

Two windows-related issues to be solved:

  1. working path in C driver, repo path in D driver. sqlfluff throws an exception when no common path can be extracted. This is fixed in latest sqlfluff main branch via b7f6933ac52bea7f12189a54458eb0229788d107.
  2. os.path.expanduser("~") in windows-latest image does not expand and instead returns "~".