-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
Update google cloud dlp package and adjust hook and operators #29234
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.
I love the change (and the fact that we are bumping the version. But we need a comment on why we are upper-binding the version (or remove the upper-binding)
@potiuk I removed upper-bound from requirements |
static checks are failing |
610eb40
to
3e68936
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.
LGTM
@potiuk ?
Let me just rebase - just in case - I think the failures were intermittent, but just want to double-check |
Release google provider package v2023.4.3: apache/airflow#30326 apache/airflow#29234 Change-Id: I03928b0d940b710ff40d6c6de61519f99954dbee GitOrigin-RevId: eb05386e3782b85dd1bcd11127ecf2204c1751ff
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 a newsfragment file, named
{pr_number}.significant.rst
or{issue_number}.significant.rst
, in newsfragments.