-
Notifications
You must be signed in to change notification settings - Fork 40
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
✨ Add BigQuery
source and Prefect tasks
#1035
Conversation
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.
Please have a look at the responses.
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.
One last thing left with the api_connection()
method, otherwise pls create an issue to track the rest and link it in relevant conversations so I can close them, and then we can merge.
@angelika233 can you rebase pls, and make sure that checks pass? Otherwise, seems like this is good to go. |
ea10e2a
to
f08ff76
Compare
3f012c3
to
0106a6f
Compare
0106a6f
to
d9ff502
Compare
d9ff502
to
08fb007
Compare
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.
Added my comments
🚧 Added requirements
2fa7adc
to
6d00a97
Compare
Summary
BigQuery connector migration from viadot 1 to viadot 2.
issue: #910
Importance
Required by the migration project.
Checklist
This PR:
CONTRIBUTING.md
CHANGELOG.md