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.

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

New Metric: Review Cycle Duration within a Change Request #89

Closed
dicortazar opened this issue Oct 13, 2020 · 5 comments
Closed

New Metric: Review Cycle Duration within a Change Request #89

dicortazar opened this issue Oct 13, 2020 · 5 comments

Comments

@dicortazar
Copy link
Member

This ticket comes from a broader discussion related to responsiveness metrics.

This ticket aims at discussing the Time between Review Cycles metric.

Google doc is available at https://docs.google.com/document/d/1-86OoosdSKSthe1_K5WrlECy3j2AqdKwJEK2doF3YBU/edit?usp=sharing
@dicortazar dicortazar added the metric idea This idea is open for discussion. Ideally, fill out a detail page to focus the discussion. label Oct 13, 2020
@klumb klumb added In Discussion and removed metric idea This idea is open for discussion. Ideally, fill out a detail page to focus the discussion. labels Nov 12, 2020
@vinodkahuja
Copy link
Collaborator

We are currently working on this and the discussion is happening in the google doc.

@dicortazar
Copy link
Member Author

Just did some sanity check on the doc and went through all the suggestions. Thanks a lot!

@GeorgLink
Copy link
Member

We revised the text today and clarified the nature of the metric. Need to review again next time and then it should be ready for public review.

@geekygirldawn geekygirldawn changed the title New Metric: Time between Review Cycles New Metric: Review Cycle Duration Jan 7, 2021
@geekygirldawn geekygirldawn changed the title New Metric: Review Cycle Duration New Metric: Review Cycle Duration within a Change Request Jan 7, 2021
@geekygirldawn
Copy link
Member

Changed the title of this issue to be consistent with the new title of the metric after discussing it in the Common Metrics WG.

@ElizabethN
Copy link
Member

Merged with PR #97

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@ElizabethN @dicortazar @geekygirldawn @GeorgLink @klumb @vinodkahuja