This repository has been archived by the owner on Sep 4, 2024. It is now read-only.
Replace Airflow guide in docs/guides with Astro version of Marquez Airflow example #185
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… retitle existing guide to emphasize proxy Signed-off-by: Michael Robinson <[email protected]>
merobi-hub
changed the title
Signed-off-by: Michael Robinson <[email protected]>
Signed-off-by: Michael Robinson <[email protected]>
Signed-off-by: Michael Robinson <[email protected]>
merobi-hub
changed the title
Signed-off-by: Michael Robinson <[email protected]>
JDarDagran
approved these changes
Jun 28, 2023
Signed-off-by: Michael Robinson <[email protected]>
kentdanas
reviewed
Jun 29, 2023
|
Thanks @kentdanas for all the improvements. Incorporating them into a single forthcoming commit. |
Signed-off-by: Michael Robinson <[email protected]>
Signed-off-by: Michael Robinson <[email protected]>
Signed-off-by: Michael Robinson <[email protected]>
Signed-off-by: Michael Robinson <[email protected]>
Signed-off-by: Michael Robinson <[email protected]>
Signed-off-by: Michael Robinson <[email protected]>
wslulciuc
suggested changes
Jul 7, 2023
There was a problem hiding this comment.
Thanks for improving the docs, @merobi-hub. I have some suggestion to simplify the setup steps:
Signed-off-by: Michael Robinson <[email protected]>
|
Thanks @wslulciuc . Propose opening a separate PR to make the more substantial suggested improvements to the proxy guide so that the new quickstart is available. The separate PR will:
|
Signed-off-by: Michael Robinson <[email protected]>
|
Renamed |
wslulciuc
approved these changes
Jul 7, 2023
merobi-hub
added a commit
that referenced
this pull request
Aug 4, 2023
…rflow example (#185) * replace airflow quickstart with marquez/examples/airflow tutorial and retitle existing guide to emphasize proxy --------- Signed-off-by: Michael Robinson <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We need an easy-to-configure, functional Airflow quickstart, but airflow.md in docs/guides is not working currently and requires complex configuration across multiple services.
This:
guidesasairflow-quickstartNote: with the changes needed to upgrade to Astro this is significantly different from the original in Marquez, so it seems that both are worth maintaining.