feat: overhaul issue templates - Githubissues

ryanoasis / nerd-fonts

Iconic font aggregator, collection, & patcher. 3,600+ icons, 50+ patched fonts: Hack, Source Code Pro, more. Glyph collections: Font Awesome, Material Design Icons, Octicons, & more

https://NerdFonts.com

Other

52.6k stars 3.59k forks source link

feat: overhaul issue templates #1667

Closed gu5e closed 6 days ago

gu5e commented 1 week ago

Description

This PR overhauled the issue templates. It changed them to issue template forms and fixed mistakes and consistency.

Requirements / Checklist

[x] Read the Contributing Guidelines
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx
[x] If this contains a font/glyph add its origin as background info below (e.g. URL)
[x] Verified the license of any newly added font, glyph, or glyph set. License is: xxx

What does this Pull Request (PR) do?

This PR changes the format of the issue templates and fixes mistakes.

How should this be manually tested?

It can be manually tested by clicking the New Issue button in the Issues tab and selecting one of the categories

Any background context you can provide?

What are the relevant tickets (if any)?

Screenshots (if appropriate or helpful)

bug_report_template

feature_request_template

Finii commented 6 days ago

@allcontributors please add @gu5e for doc

allcontributors[bot] commented 6 days ago

@Finii

I've put up a pull request to add @gu5e! :tada:

Finii commented 6 days ago

I would also add these comments somehow, because frequently people do not add screenshots at all (which makes reproduction very hard) or snip just a very small fraction of the screen...

<!-- Provide screenshots where appropriate, they help tremendously -->
<!-- Try to include the title of terminal windows and possibly settings dialogues -->

Previously this section has been called 'Screenshots' explicitely, for the same reasons:

Finii commented 6 days ago

Example of people ticking all tickboxes, although it is not true...

(There was no previous discussion, it is ticked nonetheless :grimacing: )

gu5e commented 6 days ago

I have tried to address most of the things you pointed out @Finii (sorry for the accidental ticking btw, ill be sure to do it next time)

Finii commented 6 days ago

sorry for the accidental ticking btw, ill be sure to do it next time

:-D no problem at all, ppl do it all the time

Thanks for the PR, I will fix one small thing an then merge. :+1:

Finii commented 6 days ago

Thank you! :green_heart: