Create SDDS project merge request guide

Docs are here and ready for review:

https://www.drupal.org/community-initiatives/starshot-demo-design-system... →

https://www.drupal.org/community-initiatives/starshot-demo-design-system... →

Kristen Pol → credited jacobadeutsch → .

Note that Jacob has reviewed and suggesting moving the merge request creation to after the code changes which is a good idea so will do that tomorrow probably.

Some other feedback from Jacob:

1. Quotes need fixing.

 git commit -m “Added tag SDC yml file.”

2. He didn't see the yellow merge request number because the pipeline was running so update doc to reflect this.

3. It wasn't clear to him how to change the status, assigned, etc.

Step 2
2. Find the issue where you want to contribute code [example issue]

Example issue is a fixed issue and doesn't have Create issue fork

Thanks. I was just pointing at an issue that was used for this type of work… I used it when creating the documentation… perhaps I’ll remove the link to avoid confusion

Assigning back to me but this is still being reviewed by others so will hold off on changes for now.

I think removing link is best. For someone new/beginner, it can be confusing. I've been there.

I haven't contributed to merge requests before. Is there a sample issue that I can do some trials? I read through the guide and got the ideas, just want to do it by myself!

Thanks @siramsay. Will do!

@sea2709 Let me find or create an issue for you to test this out :)

Moving this back to needs work for me to make changes.

@sea2709 Please use this one to test:

https://www.drupal.org/project/demo_design_system/issues/3465333#comment... 📌 SDDS SDC Conversion Examples: Map component [MANUAL] Postponed

I have made all the changes above except for reordering when you create the MR. I'll wait on @sea2709's feedback before doing that.

Back to needs review.

@sea2709 Did you have any additional feedback based on the MR you did?

@Kristen: I just have only one comment, I think step 6 should be placed after step 15 (or at least, that is what I did). After I push changes, I will be able to create a MR.

@sea2709 Gotcha. So just the reordering of MR creation that Jacob also pointed out. I'll move back to needs work for reworking that.

@sea2709 Would you have time to rework the guide so the MR is created after the code branch is created and changes are added?

We've been doing that but I haven't had time to update the docs.

Going to call this one fixed for now. If someone wants to update to not do the draft MR, just reopen and update the docs.

Automatically closed - issue fixed for 2 weeks with no activity.