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.

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

added __init__.py file to utils #8530

Merged
merged 2 commits into from
Dec 6, 2021

Conversation

augan-rymkhan
Copy link
Contributor

@augan-rymkhan augan-rymkhan commented Dec 6, 2021

What

airbyte_cdk/utils is not recognized as a python package. It prevents to import from that.

How

add __init__.py to mark the directory as a package

Recommended reading order

  1. airbyte_cdk/utils/__init__.py

@github-actions github-actions bot added the CDK Connector Development Kit label Dec 6, 2021
@augan-rymkhan augan-rymkhan temporarily deployed to more-secrets December 6, 2021 11:38 Inactive
@augan-rymkhan
Copy link
Contributor Author

augan-rymkhan commented Dec 6, 2021

/publish-cdk dry-run=false

🕑 https://github.com/airbytehq/airbyte/actions/runs/1544475265
https://github.com/airbytehq/airbyte/actions/runs/1544475265

@augan-rymkhan augan-rymkhan temporarily deployed to more-secrets December 6, 2021 12:15 Inactive
@augan-rymkhan
Copy link
Contributor Author

augan-rymkhan commented Dec 6, 2021

/publish-cdk dry-run=false

🕑 https://github.com/airbytehq/airbyte/actions/runs/1544546675
https://github.com/airbytehq/airbyte/actions/runs/1544546675

@augan-rymkhan augan-rymkhan temporarily deployed to more-secrets December 6, 2021 12:46 Inactive
@sergei-solonitcyn sergei-solonitcyn merged commit 8b5303e into master Dec 6, 2021
@sergei-solonitcyn sergei-solonitcyn deleted the arymkhan/fix-cdk-utils-missing-init branch December 6, 2021 13:14
schlattk pushed a commit to schlattk/airbyte that referenced this pull request Jan 4, 2022
* added __init__.py file to utils

* bump the version

Co-authored-by: Auganbay <auganenu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CDK Connector Development Kit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants