Skip to content

Comments

Fix attribute error on accessing Gitbundle.repo_url#46394

Merged
jedcunningham merged 2 commits intoapache:mainfrom
astronomer:fix-repo-url-attr-error
Feb 4, 2025
Merged

Fix attribute error on accessing Gitbundle.repo_url#46394
jedcunningham merged 2 commits intoapache:mainfrom
astronomer:fix-repo-url-attr-error

Conversation

@ephraimbuddy
Copy link
Contributor

The position where we added the repo_url makes it unavailable on the GitBundle instance when the connection id is not defined. This PR fixes it and added a test to check that it has the attribute when the connection ID is not defined

ephraimbuddy and others added 2 commits February 3, 2025 22:17
The position where we added the repo_url makes it unavailable
on the GitBundle instance when the connection id is not defined.
This PR fixes it and added a test to check that it has the attribute
when the connection ID is not defined
Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
@ephraimbuddy ephraimbuddy force-pushed the fix-repo-url-attr-error branch from b0e1976 to 1c2cbae Compare February 3, 2025 21:17
@jedcunningham jedcunningham merged commit 8fa2557 into apache:main Feb 4, 2025
46 checks passed
@jedcunningham jedcunningham deleted the fix-repo-url-attr-error branch February 4, 2025 00:00
niklasr22 pushed a commit to niklasr22/airflow that referenced this pull request Feb 8, 2025
The position where we added the repo_url makes it unavailable
on the GitBundle instance when the connection id is not defined.
This PR fixes it and added a test to check that it has the attribute
when the connection ID is not defined

Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
ambika-garg pushed a commit to ambika-garg/airflow that referenced this pull request Feb 17, 2025
The position where we added the repo_url makes it unavailable
on the GitBundle instance when the connection id is not defined.
This PR fixes it and added a test to check that it has the attribute
when the connection ID is not defined

Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants