Hi folks, the December Santa Cruz DUG meeting will be all about patches! Did you know it is not a module maintainer's job to review patches? Neither did I until I found it buried on this page. It is in fact the community's responsibility's to review patches. According to the doc, "Patches need a positive review by at least two people other than the original author before being declared Reviewed and tested by the community" Trouble is, nobody reviews contrib patches, and it's a big problem. I've been contributing patches for two years and not one has ever been reviewed. I thought it would be a great idea and a service to the community for us to learn the patch review process and actually do a patch review during the December meeting.
I have a patch for us to review and since it is my patch I will be buying burritos for all reviewers. Please bring your appetite and a clean install of Drupal 8 and remember:
- Reviewing and testing patches is perhaps the easiest way for novices to contribute.
- You do not need to be a coder to help review this patch
- You definitely don't need to be a coder to test this patch
- Drupalers of all levels are welcome to the meeting
- You will definitely learn something about how Drupal works
Hope to see you there!
Comments
Thanks Paul!
It sounds great :)
Contact: https://www.hook42.com/contact
Drupal 7 Multilingual Sites: http://www.kristen.org/book
Here are a couple sites we
Here are a couple sites we will be using tonight:
D7 site: http://live-youtube-test.gotpantheon.com/
D8 site: http://www.d8test.pfvdw.com/
See you at 7:00.
Sorry I'm missing this
Sounds like a good thing to be discussing. I'm still down in So. Calif., so I'll see you all in a month.
Do you have a link to the patch
Paul,
Could you also add a link to the issue of your patch?
Thanks,
Rob
Hi Rob, here is the issue
Hi Rob, here is the issue we'll be discussing tonight:
https://drupal.org/node/2119845
Sorry you can't make it.
Thank you to everyone who
Thank you to everyone who turned out tonight and for your help with my patch. See you next month.
Update on the patch: I
Update on the patch: I addressed the comments from the meeting and posted a new patch with my comments. Would the commenters take a look and let me know if it's ready to be RTBC or not. Thanks!
Issue URL: https://drupal.org/node/2119845