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

Add issue templates #152

Merged

Conversation

tom-borcin
Copy link
Collaborator

Adding issue templates into repository. Please check the templates and let me know if the provided documentation, description, requested data or whole templates make sense.

@CLAassistant
Copy link

CLAassistant commented Feb 23, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@tore-espressif tore-espressif left a comment

Choose a reason for hiding this comment

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

Thank you very much @tom-borcin !
I left few comments that might make it even better. I'm happy to hear your ideas ;)

.github/ISSUE_TEMPLATE/other-issue.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/feature-request.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 Show resolved Hide resolved
.github/ISSUE_TEMPLATE/other-issue.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@VojtechBartoska VojtechBartoska left a comment

Choose a reason for hiding this comment

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

Small addition.

.github/ISSUE_TEMPLATE/bug-report.yml Outdated Show resolved Hide resolved
@tom-borcin tom-borcin requested review from VojtechBartoska, igrr and tore-espressif and removed request for VojtechBartoska, igrr and tore-espressif March 2, 2023 16:58
@tom-borcin
Copy link
Collaborator Author

@igrr @VojtechBartoska @tore-espressif added your suggestions, please take a look again. Thanks!

Copy link
Collaborator

@tore-espressif tore-espressif left a comment

Choose a reason for hiding this comment

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

@tom-borcin do you plan more changes or is this ready for merging?

.github/ISSUE_TEMPLATE/bug-report.yml Outdated Show resolved Hide resolved
Co-authored-by: Tomas Rezucha <[email protected]>
@tom-borcin
Copy link
Collaborator Author

@tore-espressif I accepted all suggestions, so if you have nothing else to change we can merge

@tore-espressif
Copy link
Collaborator

Failing tests unrelated to this changes -> merging

@tore-espressif tore-espressif merged commit 7181860 into espressif:master Mar 8, 2023
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.

5 participants