Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jul 3, 2023

Since we are not releasing some providers yet, we want to exclude them also from being referenced in cross-provider dependencies.


^ 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 newsfragments.

Since we are not releasing some providers yet, we want to exclude
them also from being referenced in cross-provider dependencies.
@potiuk potiuk requested review from ashb and jedcunningham as code owners July 3, 2023 10:27
@potiuk potiuk requested review from eladkal and mobuchowski July 3, 2023 10:27
@potiuk potiuk changed the title Exckude not-releaseable providers from extras not only from releasing Exclude not-releaseable providers from extras not only from releasing Jul 3, 2023
@mobuchowski
Copy link
Contributor

Looks good - the lack of openlineage in extras does not prevent it from eventually working once it's released.

@potiuk
Copy link
Member Author

potiuk commented Jul 3, 2023

Up to @eladkal if he wants to merge it :)

@eladkal eladkal merged commit bc212f8 into apache:main Jul 4, 2023
eladkal added a commit that referenced this pull request Jul 4, 2023
eladkal added a commit that referenced this pull request Jul 4, 2023
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Jul 6, 2023
@potiuk potiuk deleted the exclude-cross-providers-for-non-releasable-providers branch November 17, 2023 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants