Skip to content

[AIRFLOW-XXX] Add clarity to gcs_download_operator params#6022

Merged
potiuk merged 1 commit intoapache:masterfrom
leahecole:master
Sep 7, 2019
Merged

[AIRFLOW-XXX] Add clarity to gcs_download_operator params#6022
potiuk merged 1 commit intoapache:masterfrom
leahecole:master

Conversation

@leahecole
Copy link
Contributor

Small docs change to add clarity to the gcs_download_operator parameters

@mik-laj mik-laj added the provider:google Google (including GCP) related issues label Sep 6, 2019
@potiuk
Copy link
Member

potiuk commented Sep 6, 2019

Can you please rebase to latest apache/master @leahecole ? I think it will solve all the checks.

@leahecole leahecole force-pushed the master branch 2 times, most recently from bfc7de1 to a57a5b1 Compare September 6, 2019 15:29
Fix flake8 errors

reformat comments
@leahecole
Copy link
Contributor Author

sorry @potiuk - it's dippy lil formatting errors. I'm paying attention to it and trying to make the ci/cd pass. Meant I finally got around to setting up breeze and testing locally though!

@potiuk potiuk merged commit ce17c9b into apache:master Sep 7, 2019
@potiuk
Copy link
Member

potiuk commented Sep 7, 2019

Thanks! @leahecole for contributing. Nice to hear Breeze is being used :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants