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.

Sign up for GitHub

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

Update issue templates #6037

Merged
merged 3 commits into from
May 30, 2024
Merged

Update issue templates #6037

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
ee4265c
Update issue templates
zliu41 May 17, 2024
79d0344
Update feature request template
zliu41 May 29, 2024
a26dde3
Merge branch 'master' of github.com:IntersectMBO/plutus into zliu41/i…
zliu41 May 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 6 additions & 7 deletions .github/ISSUE_TEMPLATE/bug_report.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ body:
A clear and specific description of what the bug is.
If applicable, add screenshots, config files and/or logs to help explain the problem.
validations:
required: true
required: false
- type: textarea
id: steps-to-reproduce
attributes:
Expand All @@ -28,27 +28,27 @@ body:
2. Click on '...'
3. Scroll down to '...'
validations:
required: true
required: false
- type: textarea
id: actual-result
attributes:
label: Actual Result
description: What is the reproducible outcome?
placeholder: See error...
validations:
required: true
required: false
- type: textarea
id: expected-result
attributes:
label: Expected Result
description: A clear and concise description of what you expected to happen.
placeholder: No errors observed.
validations:
required: true
required: false
- type: textarea
id: fix-it-yourself
attributes:
label: Describe the approach you would take to fix this
label: Describe the approach you would take to fix this
description: If you would like to work on this yourself (optional!), then let us know what approach you would take so we can advise you.
validations:
required: false
Expand All @@ -61,5 +61,4 @@ body:
Version: [e.g. 20.04]
Plutus: version or commit hash
validations:
required: true

required: false
4 changes: 2 additions & 2 deletions .github/ISSUE_TEMPLATE/feature_request.yml
View file
Open in desktop
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove the enhancement label that this template adds?
Also, we discussed during the call that this probably should be more general than just a feature request. Something more general like "tell us what you think" or whatever.

Copy link
Member Author

Choose a reason for hiding this comment

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

Something more general like "tell us what you think" or whatever.

I think people can create a blank issue for that.

Copy link
Contributor

Choose a reason for hiding this comment

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

It was rather non-obvious to me that the blank issue template existed. I know it's right on that page, but it's smaller than the templates, looks differently and there are links all over the place, so I did miss it a number of times before finally finding it hiding in plain sight. Dunno, maybe it's just me.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah but I don't see a way to remove the blank issue option, and having two options for blank issues is more confusing IMO

Copy link
Contributor

Choose a reason for hiding this comment

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

Good point.

Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,11 @@ body:
label: Describe the feature you'd like
description: A clear and detailed description of what you are suggesting.
validations:
required: true
required: false
- type: textarea
id: alternatives
attributes:
label: Describe alternatives you've considered
description: A clear and concise description of any alternative solutions or features you've considered.
validations:
required: false
required: false
38 changes: 0 additions & 38 deletions .github/ISSUE_TEMPLATE/security_report.yml
View file
Open in desktop

This file was deleted.