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

Move to ros-perception org? #122

Closed
4 tasks done
130s opened this issue Apr 26, 2017 · 9 comments
Closed
4 tasks done

Move to ros-perception org? #122

130s opened this issue Apr 26, 2017 · 9 comments

Comments

@130s
Copy link
Collaborator

130s commented Apr 26, 2017


@sniekum Occasionally I have had to ask you for admin tasks, and there will still be some cases I/someone have to.

  • Do you mind giving me an admin access to this repo?
  • Alternatively, even better is to fork this to somewhere OSRF manages, e.g. ros-perception org unit.
    • Benefit is, you'll be freed from any admin chores, and the package will probably get even more visibility (despite the popular this package has already got). You'll remain the author of course.
    • Drawback is people who downloaded the source will have to adjust their git remote to the new repo's location.
@sniekum
Copy link
Owner

sniekum commented Apr 26, 2017 via email

@130s
Copy link
Collaborator Author

130s commented Apr 27, 2017

Thanks, I didn't know the limitation of a normal user repo.
Then, I prefer to "fork"ing the repo to ros-perception or somewhere under ros-* and make that as an official, so that we'll be free from the number of admins limit.

@wjwwood could you fork this repo to a GitHub org unit somewhere appropriate that OSRF maintains, and add @sniekum and @130s as admin?

@wjwwood
Copy link

wjwwood commented Apr 27, 2017

Yes sir: https://github.com/ros-perception/ar_track_alvar

130s added a commit to 130s/rosdistro that referenced this issue Apr 27, 2017
130s added a commit to 130s/rosdistro that referenced this issue Apr 27, 2017
Repo URL update decision sniekum/ar_track_alvar#122

Also this completes the necessary setup ros#14678 (review) (setup on the repo side is already done).
@130s
Copy link
Collaborator Author

130s commented Apr 27, 2017

Thanks @ wjwwood.

  • Move issues, PRs

Using github-issues-import, after successfully imported 12 issues/PRs only, the import stopped (I might have hit IQAndreas/github-issues-import#41) So I manually imported ONLY open issues 1by1 using https://github-issue-mover.appspot.com/ As a result, only 12 closed issues/merged PRs, all 25 open issues are moved, have been moved to the new repo.

  • Update dev repo in rosdistro

Done ros/rosdistro#14698

@130s 130s closed this as completed Apr 27, 2017
@130s
Copy link
Collaborator Author

130s commented Apr 27, 2017

@sniekum hopefully one last thing; could you update at the top page of this repo the summary as something like "MOVED TO https://github.com/ros-perception/ar_track_alvar"?

@130s 130s reopened this Apr 27, 2017
@carlosjoserg
Copy link

Hi guys, apologies to jump in, it might save you time.

Adding contributors with write access is feasible even for normal users.

Though I believe the best option would be to transfer ownership of the repo, which ill copy issues and PRs plus avoid confusion and it's cleaner.

Although changing the remotes is recommended, github handles all traffic forwarding very well, I have used it before. Check the last item here: https://help.github.com/articles/about-repository-transfers/

But of course all up to you!

mikaelarguedas pushed a commit to ros/rosdistro that referenced this issue Apr 27, 2017
Repo URL update decision sniekum/ar_track_alvar#122

Also this completes the necessary setup #14678 (review) (setup on the repo side is already done).
@130s
Copy link
Collaborator Author

130s commented Jun 7, 2017

@sniekum hopefully one last thing; could you update at the top page of this repo the summary as something like "MOVED TO https://github.com/ros-perception/ar_track_alvar"?

@sniekum friendly ping :)
Recently there was still a pull request against this repo.

@sniekum
Copy link
Owner

sniekum commented Jun 9, 2017 via email

@130s
Copy link
Collaborator Author

130s commented Jun 12, 2017

Thank you @sniekum!

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

4 participants