-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Add measures to reduce duplicate issues #6184
base: develop
Are you sure you want to change the base?
Conversation
Related to mermaid-js#4227 --- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/mermaid-js/mermaid/issues/4227?shareId=XXXX-XXXX-XXXX-XXXX).
|
✅ Deploy Preview for mermaid-js ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
commit: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #6184 +/- ##
=======================================
Coverage 4.47% 4.47%
=======================================
Files 383 383
Lines 54142 54142
Branches 596 596
=======================================
Hits 2425 2425
Misses 51717 51717
Flags with carried forward coverage won't be shown. Click here to find out more. |
* **bug_report.yml** - Add a mandatory checkbox for users to confirm they have searched for existing issues - Add specific questions and prompts to guide users to provide detailed information - Add a checklist reminding users to search for existing issues before creating a new one * **config.yml** - Add a more prominent reminder to search through existing issues before creating a new one * **diagram_proposal.yml, syntax_proposal.yml, theme_proposal.yml** - Add specific questions and prompts to guide users to provide detailed information - Add a checklist reminding users to search for existing issues before creating a new one
- type: checkboxes | ||
attributes: | ||
label: Confirmation | ||
description: Please confirm the following before submitting your issue. | ||
options: | ||
- label: I have searched for existing issues | ||
required: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- type: checkboxes | |
attributes: | |
label: Confirmation | |
description: Please confirm the following before submitting your issue. | |
options: | |
- label: I have searched for existing issues | |
required: true | |
- type: checkboxes | |
attributes: | |
label: Confirmation | |
description: Please confirm the following before submitting your issue. | |
options: | |
- label: I have searched for existing proposals | |
required: true |
Please move this to the top of the questions, so people won't waste time filling out the rest.
# Number of days of inactivity before an issue becomes stale | ||
daysUntilStale: 30 | ||
# Number of days of inactivity before a stale issue is closed | ||
daysUntilClose: 7 | ||
# Issues with these labels will never be considered stale | ||
exemptLabels: | ||
- Retained | ||
# Label to use when marking an issue as stale | ||
staleLabel: Inactive | ||
# Comment to post when marking an issue as stale. Set to `false` to disable | ||
markComment: > | ||
This issue has been automatically marked as stale because it has not had | ||
recent activity. It will be closed if no further activity occurs. Thank you | ||
for your contributions. | ||
If you are still interested in this issue and it is still relevant you can comment to revive it. | ||
# Comment to post when closing a stale issue. Set to `false` to disable | ||
closeComment: > | ||
This issue has been been automatically closed due to a lack of activity. | ||
This is done to maintain a clean list of issues that the community is interested in developing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're not planning on using stalebot to close issues, so you can remove this file.
state: 'open', | ||
labels: 'Status: Triage' | ||
}); | ||
const duplicate = issues.find(issue => issue.title === context.payload.issue.title); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love the idea of finding duplicates automatically, we can move this out into a separate PR, with a more powerful script, which would address some issues of this approach.
- The chance of exact matches in title would be very low.
- listForRepo would only return 30 items without pagination.
- Searching for
labels: 'Status: Triage'
would only give a subset of issues, there might be open/closed issues for the same request.
- type: checkboxes | ||
attributes: | ||
label: Confirmation | ||
description: Please confirm the following before submitting your issue. | ||
options: | ||
- label: I have searched for existing issues | ||
required: true | ||
- type: checkboxes | ||
attributes: | ||
label: Confirmation | ||
description: Please confirm the following before submitting your issue. | ||
options: | ||
- label: I have searched for existing issues | ||
required: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicated
- type: textarea | ||
attributes: | ||
label: Environment | ||
description: Provide details about your environment, such as operating system, browser, and version. | ||
placeholder: |- | ||
- Operating System: [Windows, macOS, Linux] | ||
- Browser and Version: [Chrome, Edge, Firefox] | ||
- type: textarea | ||
attributes: | ||
label: Expected Behavior | ||
description: Describe what you expected to happen. | ||
placeholder: I expected ... | ||
- type: textarea | ||
attributes: | ||
label: Actual Behavior | ||
description: Describe what actually happened. | ||
placeholder: Instead, I saw ... | ||
- type: textarea | ||
attributes: | ||
label: Additional Information | ||
description: Provide any additional information that may help us understand the issue. | ||
placeholder: Any other relevant information... | ||
- type: checkboxes | ||
attributes: | ||
label: Confirmation | ||
description: Please confirm the following before submitting your issue. | ||
options: | ||
- label: I have searched for existing issues | ||
required: true | ||
- label: I have checked the [Live Editor - Develop](https://develop.git.mermaid.live) to see if the issue is already fixed | ||
required: true | ||
- label: I have provided detailed information about the issue | ||
required: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All this is already asked in the questions above.
Is this an AI PR? :(
- type: textarea | ||
attributes: | ||
label: Use Cases | ||
description: If applicable, give some use cases for where this syntax would be useful. | ||
placeholder: The Syntax could be used for ... | ||
- type: textarea | ||
attributes: | ||
label: Syntax | ||
description: |- | ||
If possible, include a syntax which could be used to write the diagram. | ||
Try to add one or two examples of valid use-cases here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- type: textarea | |
attributes: | |
label: Use Cases | |
description: If applicable, give some use cases for where this syntax would be useful. | |
placeholder: The Syntax could be used for ... | |
- type: textarea | |
attributes: | |
label: Syntax | |
description: |- | |
If possible, include a syntax which could be used to write the diagram. | |
Try to add one or two examples of valid use-cases here. |
- type: textarea | ||
attributes: | ||
label: Use Cases | ||
description: If applicable, give some use cases for where this theme would be useful. | ||
placeholder: The theme could be used for ... | ||
- type: textarea | ||
attributes: | ||
label: Additional Information | ||
description: Provide any additional information that may help us understand the theme proposal. | ||
placeholder: Any other relevant information... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- type: textarea | |
attributes: | |
label: Use Cases | |
description: If applicable, give some use cases for where this theme would be useful. | |
placeholder: The theme could be used for ... | |
- type: textarea | |
attributes: | |
label: Additional Information | |
description: Provide any additional information that may help us understand the theme proposal. | |
placeholder: Any other relevant information... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not planning on using stalebot, this file can be removed.
Related to #4227
For more details, open the Copilot Workspace session.