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.

Sign up for GitHub

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

Jump to bottom

Remove context manager support in move() (Breaking change) #52

Merged
merged 6 commits into from
May 6, 2020
Merged

Remove context manager support in move() (Breaking change) #52

merged 6 commits into from
May 6, 2020

Conversation

vinceatbluelabs
Copy link
Contributor

@vinceatbluelabs vinceatbluelabs commented May 5, 2020 •
edited
Loading

TODO before merging:

  • PR airflow-library to stop using with in GoogleSheetsToRecordsOperator
  • PR influencers-workflows and everybodyvotes-workflows (users of GoogleSheetsToRecordsOperator) to include latest airflow-library
  • PR influencers_pipeline and meowx-workflows to stop using with

@vinceatbluelabs vinceatbluelabs requested a review from cwegrzyn May 5, 2020 02:55
@vinceatbluelabs vinceatbluelabs marked this pull request as ready for review May 5, 2020 02:55
@vinceatbluelabs vinceatbluelabs merged commit 8dfaf71 into master May 6, 2020
@vinceatbluelabs vinceatbluelabs deleted the remove_context_manager branch May 6, 2020 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bluelabsbutler bluelabsbutler bluelabsbutler left review comments

@cwegrzyn cwegrzyn Awaiting requested review from cwegrzyn

@crvena-sonja crvena-sonja Awaiting requested review from crvena-sonja

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vinceatbluelabs @bluelabsbutler