Skip to content
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

Issue 20453 - Fixes the test_http and test_sheets assert calls only #22104

Merged
merged 5 commits into from
Mar 14, 2022
Merged

Issue 20453 - Fixes the test_http and test_sheets assert calls only #22104

merged 5 commits into from
Mar 14, 2022

Conversation

harishkrao
Copy link
Contributor

The PR contains fixes for part of the issues listed in the issue #20453. It fixes the tests for the providers - Google Sheets and Simple Http only. I will be submitting more PRs for the remaining open issues for #20453.

related: #20453


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Mar 9, 2022
@harishkrao
Copy link
Contributor Author

harishkrao commented Mar 9, 2022

@potiuk @mik-laj @kaxil if you have time, can you please review the PR? If you are busy and can suggest other reviewers, that is great too. Thank you!

@harishkrao
Copy link
Contributor Author

@potiuk @mik-laj @kaxil if you have time, can you please review the PR? If you are busy and can suggest other reviewers, that is great too. Thank you!
@potiuk Just wanted to bump this up, when you have a moment.

@potiuk potiuk merged commit 3931394 into apache:main Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants