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

Replace long comments with check boxes in the Bug Report template #4378

Merged
merged 2 commits into from
Oct 4, 2020
Merged

Replace long comments with check boxes in the Bug Report template #4378

merged 2 commits into from
Oct 4, 2020

Conversation

opusforlife2
Copy link
Collaborator

What is it?

  • Bug fix (user facing)
  • Feature (user facing)
  • Code base improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • Added checkboxes to compel the user into making an effort in their bug report.

Fixes the following issue(s)

  • People seemed to ignore the long comments.

Agreement

@opusforlife2
Add checkboxes instead of long comments
9add51b 
People just seemed to ignore them.
Stypox
Stypox requested changes Oct 4, 2020
View reviewed changes
Copy link
Member

@Stypox Stypox left a comment

Choose a reason for hiding this comment

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

Looks good to me

.github/ISSUE_TEMPLATE/bug_report.md Outdated Show resolved Hide resolved
@opusforlife2 @Stypox
Use @Stypox 's suggestion
b8f7ba6 
Co-authored-by: Stypox <stypox@pm.me>
@TobiGr TobiGr added the meta Related to the project but not strictly to code label Oct 4, 2020
@TobiGr TobiGr merged commit 66a893c into TeamNewPipe:dev Oct 4, 2020
@opusforlife2 opusforlife2 deleted the update_issue_template branch October 4, 2020 13:59
This was referenced Nov 10, 2020
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TobiGr TobiGr TobiGr approved these changes

@Stypox Stypox Stypox approved these changes

@Poolitzer Poolitzer Awaiting requested review from Poolitzer

Assignees
No one assigned
Labels
meta Related to the project but not strictly to code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@opusforlife2 @TobiGr @Stypox