-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Source Tempo: migrate to lowcode + certify to beta #18936
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
Conversation
/test connector=connectors/source-tempo
Build FailedTest summary info:
|
bloked by #18931 |
airbyte-integrations/connectors/source-tempo/source_tempo/tempo.yaml
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-tempo/source_tempo/tempo.yaml
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-tempo/source_tempo/tempo.yaml
Outdated
Show resolved
Hide resolved
@davydov-d is it ok?
|
/test connector=connectors/source-tempo
Build FailedTest summary info:
|
fixed it, thanks! |
/test connector=connectors/source-tempo
Build PassedTest summary info:
|
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.
approved assuming the expected_records were created from the original implementation
/publish connector=connectors/source-tempo
if you have connectors that successfully published but failed definition generation, follow step 4 here |
* #18682 source tempo: migrate to lowcode + certify to beta * #18936 source tempo: upd changelog * #18682 source tempo: fix yaml * #18682 source tempo: rm redundant config * auto-bump connector version Co-authored-by: Octavia Squidington III <[email protected]>
What
#18682
How
Migrate to low code, set up tests, edit docs etc