Skip to content
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

update Member Expectations to include project stewardship #129

Merged
merged 1 commit into from
May 14, 2018
Merged

update Member Expectations to include project stewardship #129

merged 1 commit into from
May 14, 2018

Conversation

Trott
Copy link
Member

@Trott Trott commented May 7, 2018

This change replaces text about leadership's obligation to "protect" the
project with a hopefully-more-clear reference to project stewardship.
The use of "protect" suggests a defensive and adversarial stance.
Project stewardship, on the other hand, implies more of a caretaker
stance. It still involves ensuring the project's well-being, but does
not assume that there is a fight to be had.

@nodejs/tsc @nodejs/community-committee

@Trott
Copy link
Member Author

Trott commented May 7, 2018

(The bit about an obligation to treat others respectfully was removed because it's already in the document as "Treat all community members with respect, consideration, and highest standards of ethical conduct." and it seems to be otherwise bolted on to this paragraph. The rest of the paragraph fits in with project stewardship, but only indirectly with treating others with respect.)

TimothyGu

This comment was marked as off-topic.

mhdawson

This comment was marked as off-topic.

mhdawson

This comment was marked as off-topic.

hackygolucky

This comment was marked as off-topic.

joyeecheung

This comment was marked as off-topic.

MylesBorins

This comment was marked as off-topic.

@MylesBorins MylesBorins added tsc-agenda cc-agenda tsc-review Issues and PRs that need attention or reviews from the TSC. and removed tsc-agenda cc-agenda labels May 8, 2018
@MylesBorins
Copy link
Contributor

I've added tsc-review

keywordnew

This comment was marked as off-topic.

@mhdawson
Copy link
Member

mhdawson commented May 8, 2018

Somehow I missed adding my last comment which was that

When someone takes on a leadership role, they take on additional obligations of
project stewardship. 

does not seem to fit into the flow so I'd just remove the whole line and the paragraph break. However, I'll leave that up to you to decide if you want to do that.

@Trott
Copy link
Member Author

Trott commented May 9, 2018

@mhdawson I'd be OK with dropping that sentence as you suggest. The document as a whole works just fine without it.

bnb

This comment was marked as off-topic.

This removes text about leadership's obligation to "protect" the
project.

The use of "protect" suggests a defensive and adversarial stance.

Moreover the sentence is superfluous.
@Trott Trott merged commit 80b58f8 into nodejs:master May 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tsc-review Issues and PRs that need attention or reviews from the TSC.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants