astronomer / astro-sdk

Astro SDK allows rapid and clean development of {Extract, Load, Transform} workflows using Python and SQL, powered by Apache Airflow.
https://astro-sdk-python.rtfd.io/
Apache License 2.0
329 stars 40 forks source link

Import openleange directly #2140

Open pankajastro opened 2 months ago

pankajastro commented 2 months ago

To avoid top-level import we should directly import the OL dependency when needed

codecov[bot] commented 2 months ago

Codecov Report

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

Project coverage is 90.62%. Comparing base (162b0ab) to head (198aae0). Report is 8 commits behind head on main.

:exclamation: Current head 198aae0 differs from pull request most recent head 414b206. Consider uploading reports for the commit 414b206 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2140 +/- ## ========================================== + Coverage 89.75% 90.62% +0.87% ========================================== Files 75 74 -1 Lines 4322 4321 -1 Branches 539 538 -1 ========================================== + Hits 3879 3916 +37 + Misses 343 304 -39 - Partials 100 101 +1 ``` | [Flag](https://app.codecov.io/gh/astronomer/astro-sdk/pull/2140/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astronomer) | Coverage Δ | | |---|---|---| | [PythonSDK](https://app.codecov.io/gh/astronomer/astro-sdk/pull/2140/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astronomer) | `90.62% <100.00%> (+0.87%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=astronomer#carryforward-flags-in-the-pull-request-comment) to find out more.

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