-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
connector-templates: Cleanup connector generator #36427
Conversation
This commit cleans up connector generator, specifically: - Removes java sources and destinations from plopfile - Removes singer source from plopfile - Removes generic and python (non-http) source from plopfile - Cleans up documentation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 as far as I can understand.
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.
Please make sure @marcosmarxm is aware of this change (mainly the renaming of source-http-api
to source-python
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.
It's more aligned with our current docs. Thanks @natikgadzhi
I think Graphite got this into master with the other one. Closing. |
What
Follows up #36193 and actually removes singer from plopfile.
This commit cleans up connector generator, specifically:
Next steps
Now that the cleanup is done, next step is to go into
airbyte-ci/connectors/pipelines/build/
and figure out how to account forsrc
in templates in #35893.