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

Added a feature for bug reporting #72

Merged
merged 7 commits into from
Oct 2, 2022
Merged

Conversation

DerMoehre
Copy link
Contributor

I added a bug reporting form and a description of the bug triage.

I hope, that I understood the task correctly.

Copy link
Collaborator

@chrisbrickhouse chrisbrickhouse left a comment

Choose a reason for hiding this comment

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

This is amazing! I didn't know about this feature of GitHub, so thank you for introducing us to it. I look forward to learning more about it with you. I've reviewed the code and left feedback on particular lines with project-specific improvements. The form is great, it includes questions for all of the things that make a good bug report, but many of our users are not programmers and so our form should do more to cater to that user base.

In general, our philosophy on bug reports is that the barrier to reporting a bug should be low, otherwise users may see the form and decide to not report a bug at all. A poor bug report is better than no bug report. Even a duplicate bug report can give us helpful information. So there are some comments about making required questions optional and some wording suggestions.

I hope it's not too much feedback! I tried to be helpful and give some suggestions where I could which is why some comments look very long. If you have any questions or think I'm wrong on something, feel free to follow up in a reply. Like I said, this is an amazing start and I really appreciate this contribution!

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug_report.yml Outdated Show resolved Hide resolved
- type: markdown
attributes:
value: |
Thanks for taking the time to fill out this bug report!
Copy link
Collaborator

Choose a reason for hiding this comment

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

More of a note to myself: we should include a warning like "This report will be public. Do not include confidential research data in this report. This information might be in file names, transcripts, and some log outputs so double check before you submit!"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did not change anything here. If you want, we can add this disclaimer also into the bug report.

@DerMoehre
Copy link
Contributor Author

I changed some things in the files. I hope it was correct, as I tried to push it again. If you can't see the changes, I will try again ;-)

Copy link
Collaborator

@chrisbrickhouse chrisbrickhouse left a comment

Choose a reason for hiding this comment

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

I think you accidentally committed a configuration file from VS Code, the rest of the changes LGTM so we should be able to merge once that last thing is fixed.

Comment on lines 1 to 11
{
"folders": [
{
"path": "../../.."
},
{
"path": "../.."
}
],
"settings": {}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this a config file from VS Code? I don't think you meant to commit this. You can add *.code-workspace to your .gitignore so that git commit -a ignores files like this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you!
I added it to the .gitignore and pushed it again. I hope, it works now.
Sorry for the inconvenience

Copy link
Collaborator

Choose a reason for hiding this comment

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

No inconvenience, it's why we do code review!

@chrisbrickhouse chrisbrickhouse merged commit 3ff4da7 into JoFrhwld:dev Oct 2, 2022
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.

2 participants