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.

Sign up for GitHub

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

Jump to bottom

First step of creating an integrated airbyte engine, targetting gsheets #288

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

First step of creating an integrated airbyte engine, targetting gsheets #288

wants to merge 5 commits into from

Conversation

naswierczek
Copy link
Contributor

  • Establishes that sources and targets can support airbyte independently.
  • This currently adds a new method at the top level. THEORETICALLY we could DEFAULT to the SupportsAirbyte but I think until we can remove the feature flag, we should keep the method signature as clean as possible

@naswierczek
First step of creating an integrated airbyte engine, targetting gsheets
115661e 
- Establishes that sources and targets can support airbyte
  independently.
- This currently adds a new method at the top level. THEORETICALLY
  we could DEFAULT to the SupportsAirbyte but I think until we can
  remove the feature flag, we should keep the method signature as clean
  as possible
@naswierczek
Addressing quality
159a625
@naswierczek
Fixing tests
6101ec6
@naswierczek
More test fixing :P
62c64d6
@naswierczek
Moar test fix
b21afb3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@naswierczek