[AIRFLOW-5674][WIP] rename GoogleCloudKMSHook to CloudKmsHook#6408
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6408 +/- ##
==========================================
- Coverage 80.61% 80.05% -0.57%
==========================================
Files 626 626
Lines 36224 36228 +4
==========================================
- Hits 29203 29002 -201
- Misses 7021 7226 +205
Continue to review full report at Codecov.
|
mik-laj
left a comment
There was a problem hiding this comment.
If we use shortcuts, we should write them Capitalized e.g. GKE, GCS
https://docs.google.com/document/d/1_rTdJSLCt0eyrAylmmgYc3yZr-_h51fVlnvMmWqhCkY/edit?ts=5bb72dfd#
We need to update our planned changes
mik-laj
left a comment
There was a problem hiding this comment.
If we use shortcuts, we should write them Capitalized e.g. GKE, GCS
https://docs.google.com/document/d/1_rTdJSLCt0eyrAylmmgYc3yZr-_h51fVlnvMmWqhCkY/edit?ts=5bb72dfd#
We need to update our planned changes
|
Can you also update a note in UPDATING.md file? |
|
@mik-laj Should I change the name to CloudKMSHook? |
Part of AIP-21
Jira
Description
Tests
Commits
Documentation