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

Horizontal review #117

Closed
cwilso opened this issue Aug 25, 2023 · 5 comments · Fixed by #151
Closed

Horizontal review #117

cwilso opened this issue Aug 25, 2023 · 5 comments · Fixed by #151
Labels
fix pending There is a fix to this issue in a pending PR Project Vision Vision and Principles

Comments

@cwilso
Copy link
Collaborator

cwilso commented Aug 25, 2023

"using consistent horizontal review" is a specific methodology, not a value. (per @fantasai )

@cwilso cwilso added the Project Vision Vision and Principles label Aug 25, 2023
@cwilso cwilso self-assigned this Aug 25, 2023
@cwilso
Copy link
Collaborator Author

cwilso commented Sep 22, 2023

I agree that consistent horizontal review is a specific methodology, not a value. That's why it appears as an operational principle - operationally, we will use consistent review, to ensure broad goals in our values of a11y, i18n, etc. I think this is proper as written.

@cwilso cwilso added the Call for Input Looking for others to weigh in. label Sep 22, 2023
@frivoal
Copy link
Contributor

frivoal commented Sep 25, 2023

I think the sentence is reasonable, but the title bugs me a little. What's important about HR is not that it's consistent: you could get consistent review by doing no review at all every time, or just checking for spelling mistakes every time…

Maybe:

  • "Deliberate Review": we don't just let anybody review (though we do that do), but we have a intentional plan to review the document on some specific criteria.
  • "Cross-Functional Review": calling out that experts from various functions or disciplines participate in the assessment.
  • "Interdisciplinary Review": idem
  • "In-Depth Review": Emphasizing the depth of examination that goes beyond a surface-level review.
  • "Thorough Review": idem

In all cases, Review could be swapped for "Analysis", "Scrutiny", "Evaluation", "Assessment"

You can also take more than one adjective. E.g. "Deliberate Interdisciplinary Scrutiny"…

@dwsinger
Copy link
Contributor

how about "widely and publicly reviewed"?

@frivoal
Copy link
Contributor

frivoal commented Sep 26, 2023

that's not the same though. Horizontal review is a component of wide review, but that particular component is not particularly well described by "wide" or "public".

@dwsinger
Copy link
Contributor

I think that widely when paired with publicly like this does imply a wide internal review, and a wide internal review would imply getting input from those in other groups and disciplines. It's a mission statement, not a formal definition.

@cwilso cwilso removed their assignment Dec 20, 2023
@cwilso cwilso added fix pending There is a fix to this issue in a pending PR and removed Call for Input Looking for others to weigh in. labels Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pending There is a fix to this issue in a pending PR Project Vision Vision and Principles
3 participants