-
Notifications
You must be signed in to change notification settings - Fork 12
Adding TRANSFER.md instructions for transferring to the REC #24
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
Conversation
Thank you for writing this!
So the process is slightly more complicated:
I did not know you can transfer a repository to an org you do not belong to, though. The documentation seems a bit contradictory to me (you can transfer to anybody by writing their name and they get an email vs. you need permission to create repositories in the target org) |
@eldruin and @ryan-summers, is there anything we can do to get @eldruin's concerns addressed, and get this merged? |
Nope, I was out on vacation the last two weeks, which is why I wasn't touching it. Thanks for the reminder though, I'll review the requested changes and get this updated shortly. |
Alright, this should be ready for another read-through now. Thanks for the review (+ ping)! |
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.
Thank you for coming back to this.
Just a couple of details left.
Co-authored-by: Diego Barrios Romero <[email protected]>
Co-authored-by: Diego Barrios Romero <[email protected]>
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.
Sorry, there is a formatting mistake where the cargo owner
command block is not properly closed due to it being from a review, where github apparently does not support triple quotes.
Could you fix that manually?
Should be done. Interestingly, Github still rendered it properly with the formatting mistake. Thanks for pointing this out though, should be good now. Feel free to manually update the PR and merge if you'd like as well |
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.
Looks good to me! Thank you for following through with this.
Fixes #23 by adding a new TRANSFER.md file.