December Santa Cruz Drupal User Group Meeting - Nextspace 7pm to 9pm (Tuesday December 3, 2013)

pferlito's picture
Start: 
2013-12-03 19:00 - 21:00 America/Los_Angeles
Organizers: 

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!

Kristen Pol's picture

It sounds great :)

Here are a couple sites we

pferlito's picture

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.

Ferlito/van der Wyk -- Web design and development
pfvdw.com

Sorry I'm missing this

Torenware's picture

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

Torenware's picture

Paul,

Could you also add a link to the issue of your patch?

Thanks,
Rob

Hi Rob, here is the issue

pferlito's picture

Hi Rob, here is the issue we'll be discussing tonight:
https://drupal.org/node/2119845
Sorry you can't make it.

Ferlito/van der Wyk -- Web design and development
pfvdw.com

Thank you to everyone who

pferlito's picture

Thank you to everyone who turned out tonight and for your help with my patch. See you next month.

Ferlito/van der Wyk -- Web design and development
pfvdw.com

Update on the patch: I

pferlito's picture

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

Ferlito/van der Wyk -- Web design and development
pfvdw.com