Skip to content

Comments

Revert Asset to Dataset for Core Extension Doc#42867

Merged
kaxil merged 2 commits intoapache:mainfrom
astronomer:fix-docs
Oct 9, 2024
Merged

Revert Asset to Dataset for Core Extension Doc#42867
kaxil merged 2 commits intoapache:mainfrom
astronomer:fix-docs

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Oct 9, 2024

This PR reverts Core Extension doc change (https://airflow.apache.org/docs/apache-airflow-providers/core-extensions/index.html) from #41348

Since those docs only work on Airflow stable, we can only change this after 3.0 as those are cross-references


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

@kaxil kaxil requested a review from eladkal October 9, 2024 16:53
@kaxil kaxil requested review from Lee-W and uranusjr October 9, 2024 16:53
@kaxil kaxil merged commit bf00235 into apache:main Oct 9, 2024
@kaxil kaxil deleted the fix-docs branch October 9, 2024 18:45
@kaxil
Copy link
Member Author

kaxil commented Oct 9, 2024

Merging regardless of failures since the failure seem to be unrelated to Assets:

############################## Start docs build errors summary ##############################
============================== apache-airflow-providers ==============================
------------------------------ Error   1 --------------------
 WARNING: unknown document: 'apache-airflow-providers-edge:configurations-ref'
File path: /opt/airflow/docs/apache-airflow-providers/<unknown>
------------------------------ Error   2 --------------------
 WARNING: unknown document: 'apache-airflow-providers-opensearch:configurations-ref'
File path: /opt/airflow/docs/apache-airflow-providers/<unknown>
------------------------------ Error   3 --------------------
 WARNING: unknown document: 'apache-airflow-providers-opensearch:logging/index'
File path: /opt/airflow/docs/apache-airflow-providers/<unknown>
------------------------------ Error   4 --------------------
 WARNING: unknown document: 'apache-airflow-providers-standard:operators'
File path: /opt/airflow/docs/apache-airflow-providers/<unknown>
------------------------------ Error   5 --------------------
 WARNING: unknown document: 'apache-airflow-providers-snowflake:operators/snowpark'
File path: /opt/airflow/docs/apache-airflow-providers/<unknown>
############################## End docs build errors summary ##############################

kunaljubce pushed a commit to kunaljubce/airflow that referenced this pull request Oct 13, 2024
pavansharma36 pushed a commit to pavansharma36/airflow that referenced this pull request Oct 14, 2024
R7L208 pushed a commit to R7L208/airflow that referenced this pull request Oct 17, 2024
harjeevanmaan pushed a commit to harjeevanmaan/airflow that referenced this pull request Oct 23, 2024
PaulKobow7536 pushed a commit to PaulKobow7536/airflow that referenced this pull request Oct 24, 2024
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants