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

chore: add reproduire actions #178

Merged
merged 1 commit into from
Feb 23, 2024
Merged

Conversation

BobbieGoede
Copy link
Member

πŸ”— Linked issue

❓ Type of change

  • πŸ“– Documentation (updates to the documentation, readme or JSDoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

With these actions a comment asking for a reproduction and explaining why should be added automatically whenever the needs reproduction label is added, and if the label is not removed within 7 days the issue should get closed automatically and labeled with closed by bot. I created an issue on my fork to demonstrate here: BobbieGoede#5.

Often times users open issues without a reproduction and remain unresponsive when asking for a reproduction, or refuse to provide one because it's simple to reproduce in their eyes. This will make checking, asking for reproductions and closing these issues less of a chore (I also think people are less likely to take it personally if it's automated).

I'll be going through the issues to add needs reproduction where applicable after this is merged, so may receive a bunch of notifications. If we have some reproduction templates we can add these to the automated comment (as well as the PR template).

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have added tests (if possible).
  • I have updated the documentation accordingly.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 23, 2024
@BobbieGoede BobbieGoede self-assigned this Feb 23, 2024
Copy link
Collaborator

@cpreston321 cpreston321 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I think this will be a great addition!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 23, 2024
@BobbieGoede BobbieGoede merged commit 6695d58 into vueuse:main Feb 23, 2024
5 checks passed
@BobbieGoede BobbieGoede deleted the chore/reproduire branch February 23, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants