Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate executor docs to respective providers #37728

Merged
merged 7 commits into from
Feb 27, 2024

Conversation

RNHTTR
Copy link
Contributor

@RNHTTR RNHTTR commented Feb 26, 2024

Closes: #33916

@RNHTTR
Copy link
Contributor Author

RNHTTR commented Feb 26, 2024

Accidentally ran my commit with --no-verify... please hold!

@RNHTTR
Copy link
Contributor Author

RNHTTR commented Feb 27, 2024

Seems like this test is failing in at least a couple PRs

  • Tests / Docker-compose quick start with PROD image verifying (pull_request)

@potiuk potiuk force-pushed the new-migrate-executor-docs-to-providers branch from 337030f to 80bc305 Compare February 27, 2024 13:18
@potiuk
Copy link
Member

potiuk commented Feb 27, 2024

Yes. All those that have not been rebased after #37713

@potiuk
Copy link
Member

potiuk commented Feb 27, 2024

(I just rebased yours)

@potiuk
Copy link
Member

potiuk commented Feb 27, 2024

🤞

@potiuk
Copy link
Member

potiuk commented Feb 27, 2024

My bad.... It was a mistake in main and I just merged a fix I though it was not needed initially #37739 🤦 .... rebasing again

@potiuk potiuk force-pushed the new-migrate-executor-docs-to-providers branch from e8d045c to 9a4b6b0 Compare February 27, 2024 14:06
@potiuk potiuk merged commit cd33c2a into apache:main Feb 27, 2024
64 checks passed
@potiuk
Copy link
Member

potiuk commented Feb 27, 2024

Wooo ohoooooo

@RNHTTR
Copy link
Contributor Author

RNHTTR commented Feb 27, 2024

omg finally!! 🥳

@RNHTTR RNHTTR deleted the new-migrate-executor-docs-to-providers branch February 27, 2024 19:06
@potiuk
Copy link
Member

potiuk commented Feb 27, 2024

Well... Let's see in main and then when we release the docs :)

abhishekbhakat pushed a commit to abhishekbhakat/my_airflow that referenced this pull request Mar 5, 2024
@ephraimbuddy ephraimbuddy added this to the Airflow 2.9.0 milestone Mar 6, 2024
@ephraimbuddy ephraimbuddy added the type:doc-only Changelog: Doc Only label Mar 6, 2024
utkarsharma2 pushed a commit to astronomer/airflow that referenced this pull request Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate Celery/Kubernetes Executor docs to providers
3 participants