Skip to content

Conversation

@uranusjr
Copy link
Member

When using a GitDagBundle, the user should tell us where the git repo is by either repo_url or git_conn_id. If the user provides neither, try to find a Connection named 'git_default' instead of failing directly.

If repo_url is provided, we won't try to find any Connection.

@uranusjr uranusjr force-pushed the default-to-git-default-if-nothing-provided branch 2 times, most recently from abde9f8 to be8102e Compare April 16, 2025 15:04
@kaxil kaxil force-pushed the default-to-git-default-if-nothing-provided branch from f0e2628 to af3345a Compare April 16, 2025 15:38
uranusjr and others added 3 commits April 16, 2025 21:37
When using a GitDagBundle, the user should tell us where the git repo is
by either repo_url or git_conn_id. If the user provides neither, try to
find a Connection named 'git_default' instead of failing directly.

If repo_url is provided, we won't try to find any Connection.
@uranusjr
Copy link
Member Author

Something’s wrong with dependencies…

@kaxil kaxil force-pushed the default-to-git-default-if-nothing-provided branch from af3345a to 2c475e4 Compare April 16, 2025 16:21
@kaxil
Copy link
Member

kaxil commented Apr 16, 2025

Jarek & I tested it, we are going YOLO

@kaxil kaxil merged commit dd0efe3 into apache:main Apr 16, 2025
76 of 78 checks passed
@kaxil kaxil deleted the default-to-git-default-if-nothing-provided branch April 16, 2025 16:29
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.

6 participants