At the Initiative Management discussion earlier today at DrupalCon Vienna, amateescu indicated Entity API has too few maintainers; xjm suggested that hchonov could become another Entity API maintainer. I just talked to hchonov, and he said he'd be happy to do it :)

For his job, he's already working on everything related to entities anyway, so this totally makes sense.

We should get +1s from the Entity API and Field API maintainers, especially the ones who are active lately: amateescu, Berdir and tstoeckler.

CommentFileSizeAuthor
#7 2912080-entity-7.patch505 bytestim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers created an issue. See original summary.

xjm credited hchonov.

xjm’s picture

Also we should have @hchonov comment on the issue confirming that he's read and is comfortable with the core maintainer handbook:
https://www.drupal.org/contribute/core/maintainers#maintainer-definition
and particularly the subsystem maintainer responsibilities:
https://www.drupal.org/contribute/core/maintainers#subsystem

Thanks @hchonov and thanks Wim for filing the issue!

xjm’s picture

Component: documentation » entity system
Issue tags: +Needs subsystem maintainer review, +Entity Field API
hchonov’s picture

I confirm that I've read the core maintainer handbook and I am comfortable with it.

Since I am involved with the Entity API and enjoy working with and on it I will be happy to get even more involved and I am looking forward to take on more responsibility.

Thank you all for considering me as a possible Entity API maintainer!

tstoeckler’s picture

I think this makes a lot of sense. I have learned a lot from @hchonov (not only, but in particular) about Entity API in the last years. He has been involved in numerous different parts of this mega API and continues to propel it forward in many of them. In fact, the only reason I never proposed this myself is that we have the same employer, so I want to avoid any possible sense of collusion. For that reason also not removing the "Needs subsystem maintainer review" tag and not touching the status (although I guess we need a patch first anyway). But huge +1000 from me.

tim.plunkett’s picture

Status: Active » Reviewed & tested by the community
Issue tags: -Needs subsystem maintainer review
FileSize
505 bytes
Berdir’s picture

Forgot about this before I went away for a month, +1 :)

amateescu’s picture

I just saw this issue, +1 from me as well :)

  • xjm committed 5f17a27 on 8.5.x
    Issue #2912080 by tim.plunkett, hchonov, Wim Leers, tstoeckler: Add...

  • xjm committed 57b7613 on 8.4.x
    Issue #2912080 by tim.plunkett, hchonov, Wim Leers, tstoeckler: Add...
xjm’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.5.x and 8.4.x; thanks @hchonov! I've granted you "Maintain issues" permission for core (so that you can assign issues to other maintainers, etc.) and made you a member of https://groups.drupal.org/core (so you can post announcements about the subsystem to the community). Welcome to the team. :)

hchonov’s picture

Thank you!

Status: Fixed » Closed (fixed)

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