Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented May 20, 2025

The original PR #49978 that fixed generation of proper documentation for standard provider examples added retrieval of the path by checking import location for the standard provider. Instead we are now using direct path to the provider.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label May 20, 2025
@potiuk potiuk requested a review from rawwar May 20, 2025 15:51
@rawwar
Copy link
Contributor

rawwar commented May 20, 2025

@potiuk , we also directly import system.standard in airflow-core/tests/unit/models/test_dagbag.py

Can we update it there as well?

@potiuk
Copy link
Member Author

potiuk commented May 20, 2025

Yep. It needs more fixes. will do it tomorrow, I am in ✈️ now ... Long trip :)

The original PR apache#49978 that fixed generation of proper documentation for
standard provider examples added retrieval of the path by checking
import location for the standard provider. Instead we are now using
direct path to the provider.
@potiuk potiuk force-pushed the fix-unit-tests-provider-examples branch from 54f6fe0 to 92f5c2a Compare May 21, 2025 09:10
@potiuk
Copy link
Member Author

potiuk commented May 21, 2025

OK. That one should work @rawwar -> I will look at teh test_dagbag.py

@potiuk potiuk merged commit 13c6020 into apache:main May 21, 2025
95 checks passed
@potiuk potiuk deleted the fix-unit-tests-provider-examples branch May 21, 2025 10:13
github-actions bot pushed a commit that referenced this pull request May 21, 2025
…ath (#50843)

The original PR #49978 that fixed generation of proper documentation for
standard provider examples added retrieval of the path by checking
import location for the standard provider. Instead we are now using
direct path to the provider.
(cherry picked from commit 13c6020)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@github-actions
Copy link

Backport successfully created: v3-0-test

Status Branch Result
v3-0-test PR Link

@potiuk potiuk added this to the Airflow 3.0.2 milestone May 21, 2025
potiuk added a commit that referenced this pull request May 21, 2025
…ath (#50843) (#50877)

The original PR #49978 that fixed generation of proper documentation for
standard provider examples added retrieval of the path by checking
import location for the standard provider. Instead we are now using
direct path to the provider.
(cherry picked from commit 13c6020)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request May 21, 2025
…50843)

The original PR apache#49978 that fixed generation of proper documentation for
standard provider examples added retrieval of the path by checking
import location for the standard provider. Instead we are now using
direct path to the provider.
dadonnelly316 pushed a commit to dadonnelly316/airflow that referenced this pull request May 26, 2025
…50843)

The original PR apache#49978 that fixed generation of proper documentation for
standard provider examples added retrieval of the path by checking
import location for the standard provider. Instead we are now using
direct path to the provider.
kaxil pushed a commit that referenced this pull request Jun 3, 2025
…ath (#50843) (#50877)

The original PR #49978 that fixed generation of proper documentation for
standard provider examples added retrieval of the path by checking
import location for the standard provider. Instead we are now using
direct path to the provider.
(cherry picked from commit 13c6020)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
…50843)

The original PR apache#49978 that fixed generation of proper documentation for
standard provider examples added retrieval of the path by checking
import location for the standard provider. Instead we are now using
direct path to the provider.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants