• Clarify long-term maintenance of the Configuration system, Configurables, and Configuration module in MAINTAINERS.txt.

Proposed resolution

Configuration system
- Greg Dunlap 'heyrocker' http://drupal.org/user/128537
- Daniel F. Kudwien 'sun' http://drupal.org/user/54136
- Alex Pott 'alexpott' http://drupal.org/user/157725

Configurable entity system
- Daniel F. Kudwien 'sun' http://drupal.org/user/54136
- Tim Plunkett 'tim.plunkett' http://drupal.org/user/241634

Configuration module
- ?
#1 drupal8.config-maintenance.1.patch956 bytessun
FAILED: [[SimpleTest]]: [MySQL] 40,806 pass(es), 2 fail(s), and 0 exception(s). View
Members fund testing for the Drupal project. Drupal Association Learn more


sun’s picture

Status: Active » Needs review
956 bytes
FAILED: [[SimpleTest]]: [MySQL] 40,806 pass(es), 2 fail(s), and 0 exception(s). View

Initial proposal - based on current/past contributions and a super-wild guess with regard to how "permanent" the current contributors (obviously there are MANY more!) are going to be.

Three sections, because the components are pretty independent:

- Configuration system: i.e., Drupal\Core\Config, the low-level config system itself, including stuff like the config import/export mechanism and includes/config.inc.

- Configurable entity system: i.e., Configurable* class stuff, the high-level configuration entity things, which are not actually part of the Configuration system itself, but play a key role for module APIs and the config import mechanism. This is also where potential stuff like the ctools-override/-revert handling would pile up.

- Configuration module: i.e., config.module, which will probably provide a user interface to configuration functionality.

I picked the most obvious contributors in order to get this issue/patch started - but I did NOT actually ask/verify anyone.

This is a long-term commitment, so this absolutely needs verification.

For example, I dunno whether @heyrocker actually wants to continue maintenance beyond the D8 initiative, nor are key architectural contributors like @beejeebus, @davidstrauss, @chx, and others contained (yet), since I wasn't sure about their long-term plans/intentions of involvement. Perfectly super-happy to adjust this patch! :)

The Configuration module is empty for now, as I'm not sure 1) what exact functionality config.module will actually provide in the end (considering that Configurables will probably be moved into Drupal\Core) and 2) who's going to work on the UI functionality (if any).

Status: Needs review » Needs work

The last submitted patch, drupal8.config-maintenance.1.patch, failed testing.

beejeebus’s picture

Status: Needs work » Reviewed & tested by the community

looks good to me, RTBC.

(going to assume that a failure is not caused by the patch, as it only changes maintainers.txt.)

beejeebus’s picture

i was congratulated about this issue - seems reading the issue but not the patch, it's possible to think i'm in the patch.

as this issue is about clarifying, here goes - please don't consider me for maintainers.txt, i'm far too unreliable.

tim.plunkett’s picture

Status: Reviewed & tested by the community » Needs review

Thankfully there are other members of the VDC team who have strong understandings of the configurable entity code, but I feel comfortable with being the one responsible.

But alexpott and heyrocker should +1 this as well before RTBC.

gdd’s picture

When the initiative leads were added to MAINTAINERS I agreed if it was clear that this wasn't an implicit agreement of maintenance after D8 ships. I'm unsure at this point what my involvement will be. I'm fine with it going in now, I do think it represents the state of things at this moment.

xjm’s picture

+1 from me if Alex is on board.

alexpott’s picture

Status: Needs review » Reviewed & tested by the community

Awesome :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Great! Committed and pushed to 8.x. Thanks!

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

Anonymous’s picture

Issue summary: View changes

Added current resolution