Skip to content

WG Naming: Add language evaluation framework #5207

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 22, 2020
Merged

WG Naming: Add language evaluation framework #5207

merged 2 commits into from
Oct 22, 2020

Conversation

@celestehorgan
Copy link

@celestehorgan celestehorgan commented Oct 11, 2020

Signed-off-by: Celeste Horgan celeste@cncf.io

Add the first version of the language evaluation framework.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 11, 2020
Add language evaluation framework
d01974f 
Signed-off-by: Celeste Horgan <celeste@cncf.io>
@celestehorgan
Copy link
Author

celestehorgan commented Oct 11, 2020

cc: @justaugustus @zacharysarah @jdumars

@justaugustus justaugustus mentioned this pull request Oct 12, 2020
Merged
ricochet
ricochet reviewed Oct 12, 2020
View reviewed changes
wg-naming/language-evaluation-framework.md
@@ -0,0 +1,111 @@

Copy link

@ricochet ricochet Oct 12, 2020

Choose a reason for hiding this comment

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

extra newline

wg-naming/language-evaluation-framework.md Outdated

The language evaluation framework is a guidance document developed by the Kubernetes Naming Working Group. It outlines a structured framework for evaluating language and terminology for harm to the community. This enables the community to navigate divisive conversations with a measure of clarity.

While the document was created for an open source technology project, we feel the principles outlines are applicable to other fields as well.
Copy link

@ricochet ricochet Oct 12, 2020

Choose a reason for hiding this comment

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

outlined*

wg-naming/language-evaluation-framework.md
#### Is the term militaristic?

Examples include “marshal/unmarshal”.

Copy link

@ricochet ricochet Oct 12, 2020

Choose a reason for hiding this comment

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

extra newline

wg-naming/language-evaluation-framework.md

- Language focus: is the language in use a metaphor that could be described more precisely using different words?


Copy link

@ricochet ricochet Oct 12, 2020

Choose a reason for hiding this comment

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

extra newline

ricochet
ricochet reviewed Oct 12, 2020
View reviewed changes
wg-naming/language-evaluation-framework.md

For each term under evaluation, answer all questions.


Copy link

@ricochet ricochet Oct 12, 2020

Choose a reason for hiding this comment

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

extra newline

wg-naming/language-evaluation-framework.md
#### Is the term overtly homophobic?

Examples do not include “homogenizing” or “homogenous” data.

Copy link

@ricochet ricochet Oct 12, 2020

Choose a reason for hiding this comment

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

socioeconomic status is the only category that I see as potentially missing.

Examples include adjectives that label people “the poor” vs specific designations.

@celestehorgan
Copy link
Author

celestehorgan commented Oct 13, 2020

/assign @justaugustus @zacharysarah @jdumars

/wg naming

@k8s-ci-robot k8s-ci-robot added the wg/naming Categorizes an issue or PR as relevant to WG Naming. label Oct 13, 2020
zacharysarah
zacharysarah suggested changes Oct 21, 2020
View reviewed changes
Copy link
Contributor

@zacharysarah zacharysarah left a comment

Choose a reason for hiding this comment

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

@celestehorgan Thanks for drafting this. I've made some revisions, some of which are substantial.

@spiffxp spiffxp mentioned this pull request Oct 21, 2020
Merged
@zacharysarah
Apply suggestions from code review
7818e90 
Co-authored-by: Zach Corleissen <zacharysarah@users.noreply.github.com>
@dims
Copy link
Member

dims commented Oct 22, 2020

/approve

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 22, 2020
spiffxp
spiffxp approved these changes Oct 22, 2020
View reviewed changes
Copy link
Contributor

@spiffxp spiffxp left a comment

Choose a reason for hiding this comment

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

/hold
to be removed at wg-naming approvers' discretion

/lgtm
I didn't have any detailed comments to add. I found this useful to reference in a PR recommending alternatives to 'master' #5243

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 22, 2020
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 22, 2020
@justaugustus
Copy link
Member

justaugustus commented Oct 22, 2020

Great work, @celestehorgan, and thanks to everyone for the reviews!
/lgtm
/approve
/hold cancel

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 22, 2020
@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Oct 22, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: celestehorgan, dims, justaugustus, spiffxp

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit 84d14fd into kubernetes:master Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dims dims Awaiting requested review from dims

@parispittman parispittman Awaiting requested review from parispittman

3 more reviewers

@ricochet ricochet ricochet left review comments

@zacharysarah zacharysarah zacharysarah requested changes

@spiffxp spiffxp spiffxp approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@spiffxp spiffxp

@justaugustus justaugustus

@jdumars jdumars

@zacharysarah zacharysarah

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. wg/naming Categorizes an issue or PR as relevant to WG Naming.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@celestehorgan @dims @justaugustus @k8s-ci-robot @spiffxp @ricochet @zacharysarah @jdumars