-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
🎉 Source Google Workspace Admin Reports: migrate to the CDK #7623
🎉 Source Google Workspace Admin Reports: migrate to the CDK #7623
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.
LGTM
…s_refactor_to_use_the_cdk
/test connector=source-google-workspace-admin-reports
|
/test connector=source-google-workspace-admin-reports
|
/test connector=source-google-workspace-admin-reports
|
/publish connector=connectors/source-google-workspace-admin-reports
|
What
Refactoring Source Google Workspace Admin Reports to use the CDK
Closes #7531
How
This task is part of issue #6164. The Source significantly relies on dead CDK code. So import BaseClient and BaseSource from base_python were replaced by import from deprecated package of the CDK. Gradle file, setup.py, Dockerfile, requirements.txt were updated. main_dev.py was renamed to main.py, updated, and added to .dockerignore.
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog example