Skip to content

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 4 commits into from
Aug 12, 2025
Merged
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
40 changes: 40 additions & 0 deletions meetings/2025-08-04.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
# 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
- 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.
- 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**
Loading