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

Feature: Clarify Interpretation of PR Velocity and PR Overview Elements #194

Closed
2 tasks done
dev-phantom opened this issue Oct 17, 2023 · 9 comments
Closed
2 tasks done
Labels
💡 feature A label to note if work is a feature 👀 needs triage

Comments

@dev-phantom
Copy link

Type of feature

📝 Documentation

Current behavior

The documentation for PR Velocity and the PR Overview is currently missing crucial context related to the elements displayed, and it's important to provide users with a comprehensive understanding of these elements. The current documentation should be updated to reflect the elements displayed in the PR Velocity and PR Overview correctly.

Original Text:
(Include the section mentioning PR Velocity from the screenshot)
https://docs.opensauced.pizza/community/pr-velocity/

Additional Context:
Enhancing this documentation will ensure that users can accurately interpret and benefit from the information presented in both the PR Velocity and PR Overview, making it a more user-friendly tool for assessing the state of ongoing and completed contributions.

Suggested solution

Revised Text:
(Include the same section but add the following clarifying paragraph)

"In the context of the PR Velocity, when you see a value like "66% in 10 days," it signifies that within a period of 10 days, 66% of the PRs (Pull Requests) were successfully merged out of the total PR count. This indicates the efficiency and speed of the pull request workflow.

In the PR Overview, it displays the following key information:

  • "6 PRs" indicates the total number of pull requests in the repository.
  • The green line represents the number of open pull requests.
  • The red line represents the number of closed pull requests.
  • The purple line represents the number of merged pull requests.
  • Total percentage of pull requests" summarizes the open, closed, and merged PRs relative to the total number of pull requests in the repository.

Understanding these elements is essential for users to quickly assess the status of pull requests and contributions within the repository."

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@dev-phantom dev-phantom added 👀 needs triage 💡 feature A label to note if work is a feature labels Oct 17, 2023
@dev-phantom
Copy link
Author

@CBID2

@CBID2
Copy link
Contributor

CBID2 commented Oct 17, 2023

@CBID2

You want to work on this @dev-phantom?

@dev-phantom
Copy link
Author

dev-phantom commented Oct 17, 2023

@CBID2

You want to work on this @dev-phantom?

Nope
Although I don't think it should take time to fix

@CBID2
Copy link
Contributor

CBID2 commented Oct 17, 2023

Ok, hey @BekahHW, I want to implement this information but I want to hear your perspective before I submit a PR.

@CBID2 CBID2 self-assigned this Oct 17, 2023
@BekahHW
Copy link
Member

BekahHW commented Oct 17, 2023

I think we're going to take care of this on the app side of things. I'll open up the issue over there and link to this issue.

@dev-phantom
Copy link
Author

dev-phantom commented Oct 17, 2023

@BekahHW please can you reopen the issue and close it as not planned as this issue wasn't fix or fix here instead of closing it as completed

@BekahHW
Copy link
Member

BekahHW commented Oct 17, 2023

The issue is linked in the app repo. This is not an issue for docs.

@dev-phantom
Copy link
Author

The issue is linked in the app repo. This is not an issue for docs.

That's correct. The issue mentioned is linked to the app repository, not the documentation. When I mentioned that it should be closed as 'not planned,' I meant that this specific feature or task was not part of the plan for this particular repository. It hasn't been implemented here and won't be, which is why i suggest this issue should be closed as 'not planned' rather than 'completed.'

@CBID2 CBID2 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 17, 2023
@CBID2
Copy link
Contributor

CBID2 commented Oct 17, 2023

The issue is linked in the app repo. This is not an issue for docs.

That's correct. The issue mentioned is linked to the app repository, not the documentation. When I mentioned that it should be closed as 'not planned,' I meant that this specific feature or task was not part of the plan for this particular repository. It hasn't been implemented here and won't be, which is why i suggest this issue should be closed as 'not planned' rather than 'completed.'

Done @dev-phantom

@CBID2 CBID2 removed their assignment Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 feature A label to note if work is a feature 👀 needs triage
Projects
None yet
Development

No branches or pull requests

3 participants