Draft of WASMEdge governance review. #772
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Josh Berkus <[email protected]>
TheFoxAtWork
approved these changes
Feb 18, 2025
|
Hi @jberkus, I'm wondering whether I should address the 'Miscellaneous issues' now or wait until the review is complete. Let me know what do you think. Thanks. |
xmulligan
reviewed
Feb 20, 2025
|
|
||
| - [mailing list](https://groups.google.com/g/wasmedge/): rarely used | ||
| - [Discord](https://discord.com/invite/JHxMj9EQbA): very active | ||
| - [community meeting](https://docs.google.com/document/d/1iFlVl7R97Lze4RDykzElJGDjjWYDlkI8Rhf8g4dQ5Rk/edit?tab=t.0: held regularly, videos uploaded |
There was a problem hiding this comment.
Suggested change
| - [community meeting](https://docs.google.com/document/d/1iFlVl7R97Lze4RDykzElJGDjjWYDlkI8Rhf8g4dQ5Rk/edit?tab=t.0: held regularly, videos uploaded | |
| - [community meeting](https://docs.google.com/document/d/1iFlVl7R97Lze4RDykzElJGDjjWYDlkI8Rhf8g4dQ5Rk/edit?tab=t.0:) held regularly, videos uploaded |
xmulligan
reviewed
Feb 20, 2025
|
|
||
| Status: Mostly Satisfactory | ||
|
|
||
| WASMEdge has clearly made progress in all aspects of governance and contributor management since joining the CNCF. They have many miscellaneous issues with governance and contributor documentation, which is normal for a project at this stage. The project needs to use its advancement to Incubating in order to drive more contributor recruitment. |
There was a problem hiding this comment.
Suggested change
| WASMEdge has clearly made progress in all aspects of governance and contributor management since joining the CNCF. They have many miscellaneous issues with governance and contributor documentation, which is normal for a project at this stage. The project needs to use its advancement to Incubating in order to drive more contributor recruitment. | |
| WASMEdge has clearly made progress in all aspects of governance and contributor management since joining the CNCF. They have many miscellaneous issues with governance and contributor documentation, which is normal for a project at this stage. The project needs to use its advancement to Incubating in order to drive more contributor recruitment. |
xmulligan
reviewed
Feb 20, 2025
| Do a quick sanity check on the number of maintainers. A very large critical project applying for graduation which has three maintainers is probably at bus-factor risk. In contrast, a narrowly-scoped project with 50 maintainers is probably listing people who are not actually maintainers. | ||
| Also check that maintainers on the list are active using Devstats, GitHub and LFX Insights. Query inactive maintainers. ---> | ||
|
|
||
| Currently WASMEdge has [four Maintainers](https://github.com/WasmEdge/WasmEdge/blob/master/docs/OWNER.md), which seems reasonable for a project this size. They are also the foudning maintainers of WASMEdge, and there has yet to be any maintainer turnover. All are active. The project has promoted many regular contributors to intermediate levels of code ownership (Reviewer and Committer), and seems prepared to eventually advance one or more contributors to maintainer according to the documented requiremens. |
There was a problem hiding this comment.
Suggested change
| Currently WASMEdge has [four Maintainers](https://github.com/WasmEdge/WasmEdge/blob/master/docs/OWNER.md), which seems reasonable for a project this size. They are also the foudning maintainers of WASMEdge, and there has yet to be any maintainer turnover. All are active. The project has promoted many regular contributors to intermediate levels of code ownership (Reviewer and Committer), and seems prepared to eventually advance one or more contributors to maintainer according to the documented requiremens. | |
| Currently WASMEdge has [four Maintainers](https://github.com/WasmEdge/WasmEdge/blob/master/docs/OWNER.md), which seems reasonable for a project this size. They are also the founding maintainers of WASMEdge, and there has yet to be any maintainer turnover. All are active. The project has promoted many regular contributors to intermediate levels of code ownership (Reviewer and Committer), and seems prepared to eventually advance one or more contributors to maintainer according to the documented requirements. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the draft copy of the WASMEdge review, in response to #690 .
This is the first review in the new format. It's somewhat ad-hoc because the review request, and Incubading request, was not in the current checklist format.
attn: @geekygirldawn @edrob999 @xmulligan
assign @TheFoxAtWork @dzolotusky @dims