backdrop-ops / contrib

Apply to join the contributed code developer team.
23 stars 16 forks source link

Add info about what to do with a project after joining the contrib group #190

Closed alexfinnarn closed 3 years ago

alexfinnarn commented 7 years ago

I know this info was put into the issue where I applied, but I'm not seeing it here in the readme and don't know where one would find such information. Something like...

After You Join:

  1. Clean up your project as much as possible.
  2. Transfer the project to the backdrop-contrib organization following the steps mentioned in: https://help.github.com/articles/transferring-a-repository-owned-by-your-personal-account/
  3. Tag a release (link). You must tag a release in the format of "1.x-1.0.0". If you started your project with a title like "backdrop-port-of-xxx", feel free to drop the prefix or change the name to something you think makes more sense.
  4. Make sure you are subscribed to notifications for the project. If new issues are created, you need to know about them or else your project will be considered abandoned...something something...
  5. (Optional) Subscribe to the issue queue on drupal.org if your module is a port. It's always nice to know about new features and bug fixes that you can incorporate in your module as well as important security-related issues. Similarly, if you fix a major issue or add a new feature, it's nice to let the Drupal maintainer know about this.
jenlampton commented 7 years ago

Yes, this is a great idea. thanks! Would you like to make a PR against the readme and squish it in somewhere?

alexfinnarn commented 7 years ago

Okay. I added basically what I had written above. Feel free to change or add more Dad jokes :)

jenlampton commented 7 years ago

Thanks @alexfinnarn ! I've added two small comments to the PR. One other thought, since I want the 'Backdrop Contributed Project Agreement' to be high on the page, can we move this new section below it, or maybe even below 'Additional notes for non-coders' ? Otherwise it's looking good!!!

alexfinnarn commented 7 years ago

Cool. I think I've made all the changes listed in the PR.

klonos commented 7 years ago

@alexfinnarn the new PR introduces numerous instances of multiple spaces and I think breaks the 80 character limit per row.

alexfinnarn commented 7 years ago

sorry, I was trying to edit via Github UI which obscures such things. PR updated.

klonos commented 7 years ago

@alexfinnarn what I tend to do is copy the text to a text editor that has a marker for the 80 characters limit (you can also consider one that has spell checking), I edit the text there, and when I'm ready, I copy/paste it back on GitHub 😉

I approved the changes and also made a recommendation of how the paragraph about subscribing to the d.org issue queue should be better worded. Please review and let me know what you think about it. Thanx.

ghost commented 7 years ago

I'm in the process of transferring a module to backdrop-contrib, and Github's asking me about team access:

Please select any teams you wish to have access to backdrop-contrib/featherlight.

I'll try to work out what to do here, but might be good to add this to the proposed steps...

jenlampton commented 7 years ago

That's a great point @BWPanda, please let us know if you need help figuring out what to do! Once we know I would love to get it added to the documentation :)

ghost commented 7 years ago

I just ticked both options ('authors' and 'security' IIRC). Still trying to figure out how to change from 'master' branch to '1.x-1.x' and then make a release...

jenlampton commented 7 years ago

@BWPanda I've added a more detailed answer over at https://github.com/backdrop-ops/contrib/issues/234. Would you be willing to create a new PR for this repo with the steps you followed?

This is what we recommend for permissions when transferring a project into the contrib group: Team: Authors - Admin access Team: Security - Write access Yourself, with Admin access (just in case!)

jenlampton commented 7 years ago

Re-opening since we need more updates :)

ghost commented 7 years ago

@jenlampton I never saw an option to add myself as an admin... And I can't create a screenshot without transferring another repository over.

jenlampton commented 7 years ago

It's a blank "Collaborators" text field at the very bottom of the page, but when you type into it it autocompletes. Not very intuitive... Maybe this would make a good video? I'll give that a shot :)

ghost commented 7 years ago

Here's what I see when I transfer a repository:

screenshot-2017-9-11 bwpanda dummy

No option to select myself as an admin...

Please now delete that repo: https://github.com/backdrop-contrib/dummy

jenlampton commented 7 years ago

No option to select myself as an admin...

It's not in this section, as this is for "Teams". The "Collaborators" section should be immediately below.

One thing to note is that you are a member of the "Authors" team so by transferring a project from yourself to Authors you are not necessarily removing your own access - as long as you grant Authors "Admin" privileges when you transfer.

After you check those two boxes, does it let you choose read/write/admin?

Please now delete that repo: https://github.com/backdrop-contrib/dummy

deleted :) wanna try it again?

ghost commented 7 years ago

Here is the entire process that I see. Hope this helps explain things...

peek 2017-09-12 09-26

Can delete this again, thanks: https://github.com/backdrop-contrib/dummy2

jenlampton commented 7 years ago

Sigh, yes, this is a problem :(

We have a bot that @Gormartsen has been working on that is supposed to "fix" the permissions on transferred repositories after the fact, but AFAIK that hasn't been set up yet.

Until that starts working feel free to open an issue in this queue for any project you need to have fixed. (And maybe we should add that to our new-maintainer notes as well, until we get the bot going.)

ghost commented 7 years ago

Ok, good to know what the issue is. I'll make a PR with updated instructions then.

jenlampton commented 7 years ago

Ok, good to know what the issue is.

I'm not sure we've sorted out the issue yet, but we do at least know it's a problem and are planning on fixing it using the GitHub API, since apparently the UI can't handle it. :(

I did make one change to the group settings after watching this video, but I can't test it since I'm the organization owner... hm. Maybe I can test it as the GitHub bot? ...

ghost commented 7 years ago

If you need me to try transferring again to test anything, let me know.

jenlampton commented 7 years ago

ooh! that worked for dragonbot! Can you try it again?

ghost commented 7 years ago

That worked! Same as last time (see above video), but this time the Settings option was available once I visited the transferred repo. I was listed as an admin, however Authors and Security only had Read access (had to manually change to Admin and Write respectively).

https://github.com/backdrop-contrib/dummy3

jenlampton commented 7 years ago

Well, one step in the right direction :) I'm not sure how to set the intial access level for the teams... I looked all over and couldn't find anything. Maybe we'll need the bot for that.

ghost commented 7 years ago

Looks like these still need deleting: https://github.com/backdrop-contrib/dummy3 https://github.com/backdrop-contrib/egg

jenlampton commented 7 years ago

deleted, thanks! :)

ghost commented 6 years ago

Any thoughts on my PR above?

jenlampton commented 6 years ago

Merged!