See #2807237: PHPUnit initiative.

klausi and dawehner want to do it.

CommentFileSizeAuthor
#2 2808121-2.patch442 bytesklausi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

klausi created an issue. See original summary.

klausi’s picture

Status: Active » Needs review
Parent issue: » #2807237: PHPUnit initiative
FileSize
442 bytes

Patch.

alexpott’s picture

Assigned: Unassigned » Dries
Status: Needs review » Reviewed & tested by the community

Looks good to me.

xjm’s picture

I think Dries might want to see the plan and roadmap for the initiative before adding it to the governance. (What we talked about on Monday.) Looks like it's being developed here: #2807237: PHPUnit initiative

But maybe we should get that signed off first.

klausi’s picture

Assigned: Dries » Unassigned
Status: Reviewed & tested by the community » Postponed

Sure, we can also assign the main initiative issue to Dries first.

xjm’s picture

Status: Postponed » Needs review

So this is now an active initiative! #2807237: PHPUnit initiative (Edit: Well it was totally active already, but now it's official too.)

@dawehner and @klausi, can you review the description of the initiative coordinator role described in https://www.drupal.org/contribute/core/maintainers#initiative and make sure that matches with your expectations?

After that the next step will be for Dries to review and see if he considers the role a good fit also.

dawehner’s picture

Works for me.

klausi’s picture

Assigned: Unassigned » Dries
Status: Needs review » Reviewed & tested by the community

Yes, the initiative coordinator role works for me.

Thanks for mediating the initiative to Dries for us xjm!

Assigning to Dries then is the next step I assume?

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed 41335f2 to 8.3.x and 219813a to 8.2.x. Thanks!

  • alexpott committed 41335f2 on 8.3.x
    Issue #2808121 by klausi, xjm, dawehner: Add PHPUnit initiative to...

  • alexpott committed 219813a on 8.2.x
    Issue #2808121 by klausi, xjm, dawehner: Add PHPUnit initiative to...
alexpott’s picture

Status: Fixed » Reviewed & tested by the community

Oops I missed that this was assigned to Dries to approve the initiative coordinators. I jumped the proverbial gun. Sorry everyone. And thanks @xjm for pointing this out.

  • alexpott committed 51bac45 on 8.3.x
    Revert "Issue #2808121 by klausi, xjm, dawehner: Add PHPUnit initiative...

  • alexpott committed 051e3c6 on 8.2.x
    Revert "Issue #2808121 by klausi, xjm, dawehner: Add PHPUnit initiative...
Dries’s picture

Oops. +1 to klausi and dawehner being the maintainers.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Thanks @Dries.

Committed and pushed 5d97345 to 8.3.x and 4f21b0a to 8.2.x. Thanks!

  • alexpott committed 5d97345 on 8.3.x
    Issue #2808121 by klausi, xjm, dawehner: Add PHPUnit initiative to...

  • alexpott committed 4f21b0a on 8.2.x
    Issue #2808121 by klausi, xjm, dawehner: Add PHPUnit initiative to...

Status: Fixed » Closed (fixed)

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