-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
🎉 Native Gitlab connector #4027
🎉 Native Gitlab connector #4027
Conversation
/test connector=connectors/source-gitlab
|
/test connector=connectors/source-gitlab
|
airbyte-integrations/connectors/source-gitlab/source_gitlab/auth.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/auth.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/streams.py
Outdated
Show resolved
Hide resolved
/test connector=connectors/source-gitlab
|
/test connector=connectors/source-gitlab
|
airbyte-integrations/connectors/source-gitlab/source_gitlab/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/source.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/streams.py
Outdated
Show resolved
Hide resolved
airbyte-integrations/connectors/source-gitlab/source_gitlab/source.py
Outdated
Show resolved
Hide resolved
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.
agreed to pick up ticket and make fixes myself
/test connector=connectors/source-gitlab
|
…/new-source-airbyte-native-gitlab
/test connector=connectors/source-gitlab
|
/test connector=connectors/source-gitlab
|
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, but have two comments that need to be addressed... please ping if we need further discussion about it
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 once open comments are addressed
…/new-source-airbyte-native-gitlab
…/new-source-airbyte-native-gitlab
/test connector=connectors/source-gitlab
|
…/new-source-airbyte-native-gitlab
/test connector=connectors/source-gitlab
|
/publish connector=connectors/source-gitlab
|
What
closes #3694.
How
Describe the solution
Recommended reading order
x.java
y.python
Pre-merge Checklist
Expand the checklist which is relevant for this PR.
Connector checklist
airbyte_secret
in output spec./gradlew :airbyte-integrations:connectors:<name>:integrationTest
./test connector=connectors/<name>
command as documented here is passing.docs/integrations/
directory./publish
command described hereConnector Generator checklist
-scaffold
in their name) have been updated with the latest scaffold by running./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates
then checking in your changes