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.

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

Detaching from the upstream repository #53

Closed
bxsx opened this issue Jan 2, 2021 · 8 comments
Closed

Detaching from the upstream repository #53

bxsx opened this issue Jan 2, 2021 · 8 comments

Comments

@bxsx
Copy link
Contributor

bxsx commented Jan 2, 2021

It's been a long history for forking garmin-connect-export. We can see it at here: https://github.com/pe-st/garmin-connect-export/network/members

Unfortunately, it seems like @kjkjava (owner of the upstream) is not interested in the project anymore. Since 2018, we have been expecting at least a comment on the PR42@upstream kjkjava/garmin-connect-export#42
No luck. As for now, we are 271 commits ahead.

As this is the most active repository of the project, I'm suggesting to detach from the upstream.

Pros:

  • GitHub will count activity in the repository (such as commits)
  • Users will stop to be confused about multiple forks and which project is actually active
  • Contributors will know where to send PRs instead of randomly sending them around forks
    Cons:
  • Breaking a great history of forking the project ;)

Here are some docs regarding the issue:

@pe-st
Copy link
Owner

pe-st commented Jan 3, 2021

I agree that it is long due to detach from @kjkjava's repo. The procedure basically is:

To detach the fork and turn it into a standalone repository on GitHub, contact GitHub Support or GitHub Premium Support. If the fork has forks of its own, let GitHub Support know if the forks should move with your repository into a new network or remain in the current network.

As there are about 30 forks of my own repo, I'll hope I get some feedback of the owners listed below if they want to "move with my (@pe-st) repository or remain in the network of kjkjava".

So I'll wait until end of January. I'll assume that repo owners listed below want to keep the "fork" relation with my (@pe-st) repo unless they react with an email, a comment or a 👎🏽 (-1) emoji to this message.

If you want to stay a fork of @kjkjava then please react with a 👎🏽 until January 31, if you want to move with this repo and stay a fork of @pe-st please react with a 👍🏽

So here's the list of the forks of my fork; I hope everybody gets notified when mentioned here:

@alcantarar @amitra @andyakrn @benhollomon @chrismetcalf @chs8691 @rb83421 @cfculhane @cristian5th @gmajor67 @hrpomrx @ibury08 @jedie @jkall @jonasbg @jtorrex @lindback @mdrumond @meinert @mipapo @realflunschi @reto @roballey @rsjrny @storopoli @telemaxx @thetiger7 @wscott1541 @zeair

@roballey
Copy link

roballey commented Jan 3, 2021 via email

@cfculhane
Copy link

Great idea, thanks for pursuing this!

@rb83421
Copy link

rb83421 commented Jan 3, 2021 via email

@jkall
Copy link
Contributor

jkall commented Jan 4, 2021 via email

@bxsx
Copy link
Contributor Author

bxsx commented Feb 7, 2021

Looks like we can move forward. Did you get any disagreement in private, @pe-st ?

@pe-st
Copy link
Owner

pe-st commented Feb 10, 2021

The repo has been detached on 2021-02-03

@pe-st pe-st closed this as completed Feb 10, 2021
@kjkjava
Copy link
Contributor

kjkjava commented Feb 25, 2021

@pe-st Thank you for taking ownership of this project. As you (and all other contributors) have noticed, I am currently unable to keep up with this project. I feel bad for not at least responding to issues/PRs, but I've been pulled in many directions with work and other various projects.

I'm happy to add others to my repo to maintain it, but it sounds like all I need to do at this point is point others to your fork, right?

I'm glad Garmin finally added the ability to export data in bulk, but I will have to give your updated tool a try soon for the added features!

jonasbg added a commit to jonasbg/garmin-connect-export that referenced this issue Feb 26, 2021
* Mention the detachment from the upstream repo

pe-st#53

* Blind fix for the http error 402 by adding nk: NT to the header

* Update CHANGELOG and README for 3.0.3

Co-authored-by: Peter Steiner <unistein+pitchounette@gmail.com>
Co-authored-by: Reto Conconi <reto@users.noreply.github.com>
Co-authored-by: Reto Conconi <reto.conconi@borm.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants