-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
Add blog post publishing guidelines #7860
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -88,3 +88,12 @@ As seen above, the different teams under the Node.js Web Team umbrella are respo | |||||||||||||||||
Following this line of thought, the Web Infra Team is responsible for the technical aspects of the Node.js Website (Infrastructure, Framework, CI/CD, etc); The Website Team is responsible for the day-to-day development of the Node.js Website; The UX and Design Team advise on Design Matters and the Web Standards Team advise on best-practices for Web APIs and Web Technologies/Standards. | ||||||||||||||||||
|
||||||||||||||||||
But above all, the Web Team should work together to better the Web Experience for Node.js, aiming to provide the best experience for Node.js users. | ||||||||||||||||||
|
||||||||||||||||||
## Blog post publishing guidelines | ||||||||||||||||||
|
||||||||||||||||||
The following guidelines apply to all blog posts except minor or patch release announcements: | ||||||||||||||||||
|
||||||||||||||||||
1. each blog post _must_ have a target publishing date and time. If scheduled publishing is not possible, _the author_ (if privileged enough) or another delegated member would be responsible for landing. | ||||||||||||||||||
2. each blog post _must_ be approved by @nodejs/tsc members _and_ OpenJS Foundation marketing staff. | ||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm strongly -1 on having an automatic approval after any period of time. Explicit approval should be required. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
I think this is clear enough to indicate that the blog post needs at least two approvals from the reviewing teams? |
||||||||||||||||||
|
||||||||||||||||||
For minor/patch release announcements, the members of `@nodejs/releasers` can land as they see fit. | ||||||||||||||||||
mcollina marked this conversation as resolved.
Show resolved
Hide resolved
legendecas marked this conversation as resolved.
Show resolved
Hide resolved
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we can pull together a nodejs/marketing team that has members from both the TSC and the foundation marketing staff, would it be enough here to say that the nodejs/marketing team must approve it... then that team can work out whatever process makes the most sense to it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does https://github.com/orgs/nodejs/teams/social suffice?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jasnell if both the TSC and OpenJS staff are members of the team, technically the TSC could approve this without asking them. This has a different meaning.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair. If the new @nodejs/marketing team, then, is made up of Foundation marketing staff, I agree that requiring an approval from at least one person in both the @nodejs/tsc and @nodejs/marketing teams is ideal.