-
-
Notifications
You must be signed in to change notification settings - Fork 133
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
Modern templates #265
Modern templates #265
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #265 +/- ##
=======================================
Coverage 99.84% 99.84%
=======================================
Files 7 7
Lines 1181 1181
Branches 81 81
=======================================
Hits 1179 1179
Misses 1 1
Partials 1 1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sadrasabouri Thanks for your efforts 🔥
Please take a look at my comments 💯
attributes: | ||
label: Additional context | ||
placeholder: > | ||
If you have any additional context or information about the 1-line art (like the link to the source of it), please write it down here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
About the decoration
.github/ISSUE_TEMPLATE/config.yml
Outdated
contact_links: | ||
- name: Discord | ||
url: https://discord.com/invite/FAAyq3QJqP | ||
about: Ask questions and discuss with other art community members |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Art
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sadrasabouri Please take a look at my comments 🔥
placeholder: > | ||
ex: ▌│█║▌║▌║ | ||
validations: | ||
required: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this item should be true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed in 7dcf2a6.
placeholder: > | ||
ex: (っ◕‿◕)っ | ||
validations: | ||
required: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this item should be true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed in 7dcf2a6.
…rea is now required.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
What does this implement/fix? Explain your changes.
Added
decoration_request.yml
templatefont_request.yml
templateone_line_request.yml
templatefeature_request.yml
templateconfig.yml
for issue templateChanged