Skip to content

Conversation

@MaksYermak
Copy link
Contributor

@MaksYermak MaksYermak commented Feb 1, 2022

This PR fixes problems with mypy checker which were in this PR #20077 .

Co-authored-by: Wojciech Januszek januszek@google.com
Co-authored-by: Lukasz Wyszomirski wyszomirski@google.com
Co-authored-by: Maksim Yermakou maksimy@google.com


^ 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.

@jedcunningham
Copy link
Member

@MaksYermak, can you rebase on main? I think the failures you are seeing here have been fixed now.

@MaksYermak
Copy link
Contributor Author

@MaksYermak, can you rebase on main? I think the failures you are seeing here have been fixed now.

@jedcunningham I have done it

@potiuk potiuk closed this Feb 6, 2022
@potiuk potiuk reopened this Feb 6, 2022
@lwyszomi
Copy link
Contributor

lwyszomi commented Feb 8, 2022

@potiuk @jedcunningham Can we merge these operators if all is green?

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's been already reviewed and merged - this one is just after MyPy fixes. LGTM. Merging

@potiuk potiuk merged commit e973740 into apache:main Feb 8, 2022
@jedcunningham jedcunningham added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools area:providers changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) kind:documentation provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants