Add davidhernandez and mortendk as maintainers for Classy theme

Yeah, we should update maintainers.txt, either now or when (?) the whole governance re-evaluation is suppose to happen. Do we also need to add someone else to Stark? I'm not sure if John cares about babysitting it, or if it really needs any attention.

Feel free to propose individual updates to MAINTAINERS.txt. Doing a wholesale revamp of it is on my todo list, but it's not at the top. :)

I'm happy to be a maintainer of this theme, it looks like a challenge i'm ready to accept as well as working more in contributing to core patches. Looking to start dedicating more time to core and working on core every friday as well.

Thanks Morten for creating issue for this. This is important thing to decide. I don't know if you're up to this Morten but I would really like to see you as a maintainer of Classy.

I think there is multiple reasons why we should have him as a maintainer (if he wishes so). He has a great vision what Classy should be and what it is - something that I don't think any of us have. Morten has also ability to organize sprints like in Amsterdam pre-post sprints. He has been also very active in the issues related to Classy. So I wish he would be at least one of the maintainers (if multiple)!

I would like to propose davidhernandez as maintainer of the Classy theme. He's been ever-present throughout the Classy/banana implementation process. He's well organised and a good leader.

Do we also need to add someone else to Stark?

I don't consider John to be active, we definitely need someone to add or replace him. We could also discuss weather Stark needs a maintainer at all, as it's basically an empty theme. I think an active 'markup' maintainer is more pressing.

I'd be happy to do it. Classy has been my main impetus for core contribution, so there is some personal attachment there, and it is great to be recognized for the hard work. However, I don't want to overlook Morten's vision for D8 frontend, and helping get us where we are today.

I have another proposal. Since it behooves us to have more than one maintainer for each component, how about adding us both? Morten is more fiery than I am, and I like to think I'm a bit more level-headed, yet we seem to get along and work together just fine. :)

I have some more thoughts about the other theme system components, and was hoping to discuss it some during last nights Twig call, but it was just me, Joel, and Emma. If more people are available at the next one, perhaps we should set aside some time for this.

Governance is a bit boring for some, but these matters are important. We need to make sure we have good leadership and clear procedures in place, and are recognizing people for the work they are currently doing.

I was just about to comment on the issue earlier today that what about having David and Morten both. They both have reasons why they should be maintainers, Morten has a great vision and is good at hiring people to work on it, on the other hand David has also good vision and has very good skills on organizing things. So +1 for having Morten and David both as a maintainer for Classy.

We can do weekly as long as it's not me hosting every week, I can't commit to that. We were talking a few days ago on IRC about trying to set up a shared
YouTube/Google+ account.

The maintainers.txt is not intended to be for credit, so we shouldn't use it to recognise work people have already done, we have other nice ways of doing that ;-)

We should only add people as maintainers if they intend to commit to the role going forward. I think we can have two maintainers but only if they both intend to be active.

Sorry, I was commenting on #9 in particular:

it is great to be recognized for the hard work. However, I don't want to overlook Morten's vision for D8 frontend, and helping get us where we are today.

Sorry, I did not mean to imply it can be used strictly as a reward for past deeds. I agree with the sentiment that every maintainer must be active in their area.

in #3 @webchick mentioned a revamp. I think #1854480: Remove inactive maintainers from MAINTAINERS.txt is where that is being tracked.

@lauriii pointed out on IRC, the placement is alphabetical :)

Looks great!

+++ b/core/MAINTAINERS.txt
@@ -472,6 +472,10 @@ Bartik theme
+- Morten Birch Heide-Jørgensen 'mortendk' https://www.drupal.org/u/mortendk

Shouldn't that be Morten--Birch__Heide-Jørgensen? :)

+1 for both David and Morten!

+2 for both David and Morten !! :D

Woohoo!

No idea why random test fail.

Committed to 8.x. @davidhernandez and @mortendk; thanks for all your work and stepping up to being a maintainer for this theme!