search

openjs-foundation / cross-project-council

OpenJS Foundation Cross Project Council
https://openjsf.org/
MIT License
439 stars 152 forks source link

Add Ecosystem Sustainability Program (ESP) Page #1309

Closed bensternthal closed 4 months ago

bensternthal commented 4 months ago

This PR adds information about the OpenJS Ecosystem Sustainability Program, including guidelines and best practices for participating with our first partner, HeroDevs. For additional background, please see #1277

@ljharb @joesepi @tobie

I'd like to fast-track this PR.

ljharb commented 4 months ago

(separate comment for fast track thumbs-upping)

bensternthal commented 4 months ago

I'll post an update with the HeroDevs recommendations edited to be general recommendations. @tobie is right that these recommendations are not vendor specific.

If there is a requirement to ping CPC members before fast-tracking it should be added to the docs. Right now it states that if folks disagree they should down vote or remove the label.

tobie commented 4 months ago

If there is a requirement to ping CPC members before fast-tracking it should be added to the docs. Right now it states that if folks disagree they should down vote or remove the label.

There is indeed no such requirements. Imho, this PR largely exceeds what we customarily accept for fast track. I know there’s a presentation later this week for which this is needed so it felt acceptable to compromise but would require making sure people get a chance to see before it gets merged by doing outreach. Removing the fast-track label for now. Happy to allow it back in if folks think the proposed compromise is acceptable.

bensternthal commented 4 months ago

@tobie happy to wait on merging to give folks more time to review.

ovflowd commented 4 months ago

If there is a requirement to ping CPC members before fast-tracking it should be added to the docs. Right now it states that if folks disagree they should down vote or remove the label.

There is indeed no such requirements. Imho, this PR largely exceeds what we customarily accept for fast track. I know there’s a presentation later this week for which this is needed so it felt acceptable to compromise but would require making sure people get a chance to see before it gets merged by doing outreach. Removing the fast-track label for now. Happy to allow it back in if folks think the proposed compromise is acceptable.

Afaik there is a fast-tracking rule. @bensternthal this is fast-tracked, you can merge it. https://github.com/openjs-foundation/cross-project-council/blob/main/governance/GOVERNANCE.md?plain=1#L197

tobie commented 4 months ago

For the record, @ovflowd, I had removed the fast-track label (as per the line following the one you linked). I’m open to re-adding it if (1) this needs to be fast-tracked to meet the Friday deadline and (2) there’s additional outreach that’s done to make sure CPC members see it before it’s merged.