Wim has done some great work in the issue queue to provide feedback on REST module patches and he knows this subsystem well by now. Since he is already doing unofficial maintenance work let's make it explicit and add him to MAINTAINERS.txt. He is also going to work in this area in the future.

I think this is a Wim-Wim situation for Drupal, so let's add him. Thanks Wim!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

klausi created an issue. See original summary.

klausi’s picture

+1 from me as current REST module maintainer.

Wim Leers’s picture

Confirmed :)

I think this is a Wim-Wim situation for Drupal, so let's add him. Thanks Wim!

LOL :D

dipakmdhrm’s picture

FileSize
444 bytes

It would be so much fun if my patch gets into core!!!! :P

dipakmdhrm’s picture

It probably won't because the issue number was wrong in the name of the file I uploaded... :(

klausi’s picture

Status: Active » Reviewed & tested by the community

Patch looks good.

dawehner’s picture

I'm wondering what would people say when I vote for myself as well.

Wim Leers’s picture

I'm wondering what would people say when I vote for myself as well.

You also want to become REST maintainer? EDIT: confirmed in IRC, yay!

@klausi: don't we need +1 from Crell too?

klausi’s picture

Crell was fine with it in IRC when we discussed Wim as maintainer the first time, so I think we can go ahead here.

neclimdul’s picture

Don't know that this is a democratic process but Wim's done some awesome work improving REST and related routing/serialization issues recently. He deserves both the recognition and influence to push it forward afforded by being a maintainer.

+1

Crell’s picture

I'm Larry Garfield, and I approve this patch.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 3ec4d81 and pushed to 8.1.x and 8.2.x. Thanks!

  • alexpott committed 5cd36af on 8.2.x
    Issue #2718719 by dipakmdhrm: Add Wim Leers to REST module in...

  • alexpott committed 3ec4d81 on 8.1.x
    Issue #2718719 by dipakmdhrm: Add Wim Leers to REST module in...

Status: Fixed » Closed (fixed)

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