-
Notifications
You must be signed in to change notification settings - Fork 123
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
Docs: deploy airflow #410
Docs: deploy airflow #410
Conversation
…rflow as separated pages.
✅ Deploy Preview for dlt-hub-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
docs/website/docs/walkthroughs/deploy-a-pipeline/deploy-with-airflow-composer.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok this looks like quality! thanks for fixing all the relevant links also in the code!
@@ -19,7 +19,7 @@ | |||
from dlt.common.destination.reference import DestinationReference | |||
|
|||
REQUIREMENTS_GITHUB_ACTION = "requirements_github_action.txt" | |||
DLT_DEPLOY_DOCS_URL = "https://dlthub.com/docs/walkthroughs/deploy-a-pipeline" | |||
DLT_DEPLOY_DOCS_URL = "https://dlthub.com/docs/walkthroughs/deploy-a-pipeline/deploy-with-github-actions" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should have a corresponding link in our code for composer. I'll take a look in the other PR (#401 )
@@ -45,7 +45,7 @@ dlt deploy <script>.py airflow-composer --schedule "*/30 * * * *" | |||
|
|||
[Google Composer](../running-in-production/orchestrators/airflow-gcp-cloud-composer.md) is a managed Airflow deployment provided by Google. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should link IMO to your new docs page
* created folder deploy-a-pipeline, moved in there deploy github and airflow as separated pages. * added pages to sidebar, added title * fix link * fix link: walkthroughs/deploy-a-pipeline/deploy-with-github-actions.md * fix link: walkthroughs/deploy-a-pipeline/deploy-with-github-actions.md * fix folder deploy-a-pipeline * rename github actions deploy * delete md extension * rename title * rename title * fix code blocks * fix comma
created folder deploy-a-pipeline, moved in there deploy github and airflow as separated pages.