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

Clarify the Charter so that contractors are explicity counted as affialiated #1650

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

mcollina
Copy link
Member

Some people mighrt be contracted to work on OSS on behalf of companies. This does not make them less affiliated, and it might be used to circumvent the affiliation rules.

@mcollina
Copy link
Member Author

cc @nodejs/tsc

@mcollina
Copy link
Member Author

Copy link
Contributor

@ShogunPanda ShogunPanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

TSC-Charter.md Outdated Show resolved Hide resolved
Co-authored-by: Antoine du Hamel <[email protected]>
Copy link
Member

@tniessen tniessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On a side note, I would naively assume that "contractual obligations" is too broad, but I am neither a lawyer nor is English my native language.

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

with my CPC hat on

@legendecas
Copy link
Member

Not a lawyer, I assume sponsorships that bound with goals are not considered as some sort of "by any possible means, ... and contractual obligations". Is this a correct assumption?

@tniessen
Copy link
Member

@legendecas Naively, I'd assume that paying for a streaming service is a contractual obligation, but that what you are describing is not :)

Copy link

@tobie tobie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

TSC-Charter.md Outdated Show resolved Hide resolved
@mcollina
Copy link
Member Author

@richardlau @mhdawson @nodejs/tsc PTAL

Copy link
Member

@legendecas legendecas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The updated text looks better to me!

TSC-Charter.md Outdated Show resolved Hide resolved
TSC-Charter.md Outdated Show resolved Hide resolved
TSC-Charter.md Outdated
Comment on lines 41 to 42
As a result, no more than one-fourth of the TSC voting members may be affiliated with the
the same company.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this sentence can be removed if we go with my rewrite suggestion above?

Co-authored-by: Rich Trott <[email protected]>
TSC-Charter.md Outdated Show resolved Hide resolved
Signed-off-by: Matteo Collina <[email protected]>
@mcollina
Copy link
Member Author

@nodejs/tsc ptal

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ctcpip
Copy link

ctcpip commented Jan 7, 2025

The first paragraph says "company or other entity", but the 2nd paragraph refers only to "company". Consider changing those references to "company/entity" for consistency.

Copy link
Member

@joesepi joesepi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Approving as a Node.js collaborator as well as an OpenJS CPC member. ✌️

Signed-off-by: Matteo Collina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.