MeltanoLabs / tap-gitlab

Singer.io Tap for extracting data from Gitlab's API
GNU Affero General Public License v3.0
11 stars 27 forks source link

Should I use https://github.com/singer-io/tap-gitlab or https://gitlab.com/meltano/tap-gitlab ? #40

Closed pnadolny13 closed 2 years ago

pnadolny13 commented 2 years ago

In GitLab by @stephane-klein on Aug 12, 2020, 08:26

Hi,

should I use https://github.com/singer-io/tap-gitlab or https://gitlab.com/meltano/tap-gitlab ?

If I look commit history, I think that https://gitlab.com/meltano/tap-gitlab is more up to date that https://github.com/singer-io/tap-gitlab

Suggestion:

Same issue on https://github.com/singer-io/tap-gitlab repository: https://github.com/singer-io/tap-gitlab/issues/19

Best regards, Stéphane

pnadolny13 commented 2 years ago

In GitLab by @DouweM on Aug 12, 2020, 11:04

@stephane-klein https://gitlab.com/singer-io/tap-gitlab is the original and is also offered on the StitchData platform: https://www.stitchdata.com/docs/integrations/saas/gitlab. We (the Meltano team at GitLab) forked it since the Singer team has historically been slow to review and accept improvements to the repos they manage.

If you intend to use tap-gitlab with StitchData, https://github.com/singer-io/tap-gitlab is your only option. If you're planning to use Meltano or another Singer runner instead, https://gitlab.com/meltano/tap-gitlab is your best bet, since it has some additional functionality, and since we are more responsive to issues and merge requests.


Do you think it would be useful to state that explicitly in the readme?

pnadolny13 commented 2 years ago

In GitLab by @stephane-klein on Sep 30, 2020, 10:00

Do you think it would be useful to state that explicitly in the readme?

@DouweM Yes !

pnadolny13 commented 2 years ago

In GitLab by @DouweM on Sep 30, 2020, 13:06

created merge request !27 to address this issue

pnadolny13 commented 2 years ago

In GitLab by @DouweM on Sep 30, 2020, 13:06

mentioned in merge request !27

pnadolny13 commented 2 years ago

In GitLab by @DouweM on Sep 30, 2020, 13:10

mentioned in commit 26329fd9f0a4aacdcae45dae6a917fe64e7a4f38

pnadolny13 commented 2 years ago

In GitLab by @DouweM on Sep 30, 2020, 13:10

@stephane-klein Done in https://gitlab.com/meltano/tap-gitlab/-/merge_requests/27!