Skip to content

Add issue templates #391

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

Merged
merged 2 commits into from
Apr 24, 2021
Merged

Add issue templates #391

merged 2 commits into from
Apr 24, 2021

Conversation

awvwgk
Copy link
Member

@awvwgk awvwgk commented Apr 18, 2021

Two issue templates for stdlib:

  • bug report
  • feature proposal

@awvwgk awvwgk added meta Related to this repository reviewers needed This patch requires extra eyes labels Apr 18, 2021
Copy link
Member

@jvdp1 jvdp1 left a comment

Choose a reason for hiding this comment

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

Useful additions. Thanks

Copy link
Member

@LKedward LKedward left a comment

Choose a reason for hiding this comment

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

Cheers 👍

Co-authored-by: Laurence Kedward <[email protected]>
Copy link
Member

@milancurcic milancurcic left a comment

Choose a reason for hiding this comment

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

Thanks, looks good.

Is the text in html comment blocks meant to show in the issue template? I don't think it will, but maybe GitHub does something special about it.

@awvwgk
Copy link
Member Author

awvwgk commented Apr 22, 2021

The HTML comments will show up for the author when creating the issue, but will not be present in the finally submitted issue, which is therefore free from these instructions.

@awvwgk
Copy link
Member Author

awvwgk commented Apr 22, 2021

Conda-forge is using those in their issue tempates as well, checkout https://github.com/conda-forge/xtb-feedstock/issues/new

@milancurcic
Copy link
Member

Perfect, thanks, that didn't occur to me, but now that you explained it it's kinda obvious.

Copy link
Member

@ivan-pi ivan-pi left a comment

Choose a reason for hiding this comment

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

LGTM

@awvwgk awvwgk removed the reviewers needed This patch requires extra eyes label Apr 24, 2021
@awvwgk awvwgk merged commit 8228b94 into fortran-lang:master Apr 24, 2021
@awvwgk awvwgk deleted the issue-templates branch April 24, 2021 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Related to this repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants