-
-
Notifications
You must be signed in to change notification settings - Fork 3
Add 2025-08-04 minutes #34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
# Node.js Website Team Meeting 2025-08-04 | ||
|
||
## Present | ||
|
||
- Aviv Keller @avivkeller | ||
- Claudio Wunder @ovflowd | ||
- Augustin Mauroy @AugustinMauroy | ||
- flakey5 @flakey5 | ||
- Brian Muenzenmeyer @bmuenzenmeyer | ||
- Dario Piotrowicz @dario-piotrowicz | ||
- Caner Akdas @canerakdas | ||
- Matt Cowley @MattIPv4 | ||
|
||
## Agenda | ||
|
||
### nodejs/nodejs.org | ||
|
||
- https://github.com/nodejs/nodejs.org/pull/7978 | ||
- Brian: This PR changes how we handle content review. | ||
- Claudio: Historically, the website team didn't need to weigh in (since there was no real content). But that's changed with the Learn initiative. We need to, at the end of the day, protect ourselves. We shouldn't take ownership for content that we aren't experts on. | ||
- Aviv: In the previous setup, we had partial authority when no one was explicitly designated. But what if someone *is* authorized but becomes unresponsive? | ||
- Claudio: In open source, there isn't a formal concept of "authority" over content. Anyone can block a PR without introducing excessive process or bureaucracy. | ||
- Brian: Was this triggered by a specific incident? | ||
- Claudio: Yes and no. There *was* an incident, but it was a misunderstanding and fairly isolated. | ||
- Aviv: If the system isn't broken, maybe it doesn’t need to be fixed. | ||
- Brian: +1 | ||
- Claudio: +1 | ||
- Augustin: +1 | ||
- Matt: The old wording in the guidelines was intentionally vague. Clarity is also important. | ||
- Aviv: Let's continue this discussion in the PR itself. Feel free to refer to the minutes for a refresher on this conversation. | ||
|
||
- https://github.com/nodejs/nodejs.org/pull/8040 | ||
- Claudio: What if we need to add owners to many files at once? | ||
- Aviv: How often does that scenario realistically occur? | ||
- Claudio: Fairly often, for example, when people need to change pings or ownership, it becomes messy. A good system needs to be maintainable over time. | ||
- Matt: +1 in favor of introducing a `CODEOWNERS` file. | ||
- Aviv: Should we scrap the entire idea of mapping articles to Core APIs? | ||
- Claudio: No, we should just separate it from the review process. There are many ways to map files to relevant core APIs. | ||
- Augustin: A simple mapping won’t work for the Learn content. It's too complex, and we need technical writers. | ||
- Claudio: Core collaborators and TSC members are strong in technical domains but transforming technical documentation into Learn articles introduces another layer of complexity, perhaps AI will help? | ||
- **Discussion cut short, as we ran out of time** |
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.