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

Include "Feedback" in cycle time as well #32

Merged
merged 1 commit into from
Mar 2, 2024
Merged

Conversation

okurz
Copy link
Member

@okurz okurz commented Feb 27, 2024

Only counting "In Progress" yields unrealistically low cycle time
values and would not reflect overall time that a common
development ticket is being worked on within a team as often
we use "Feedback" when waiting for others within the team,
e.g. for code review.

This commit counts time in status "Feedback" same as
"In Progress" to account for that.

Related progress issue: https://progress.opensuse.org/issues/155917

Copy link

github-actions bot commented Feb 27, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-03-02 12:07 UTC

backlogger.py Show resolved Hide resolved
Only counting "In Progress" yields unrealistically low cycle time values
and would not reflect overall time that a common development ticket is
being worked on within a team as often we use "Feedback" when waiting
for others within the team, e.g.  for code review.

This commit counts time in status "Feedback" same as "In Progress" to
account for that.

The test adaptions include the definition of "Feedback" in the mocked
data as well as a mock entry for a ticket in "Feedback" with according
adaption in the assertion but no assertion is added for the calculation
to include tickets which had transitioned over that state.

Related progress issue: https://progress.opensuse.org/issues/155917
@okurz okurz force-pushed the okurz-include-feedback branch from b76c070 to 99009f4 Compare February 28, 2024 07:56
@okurz okurz merged commit b7e0fb9 into main Mar 2, 2024
3 checks passed
@okurz okurz deleted the okurz-include-feedback branch March 2, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants