Skip to content

Enhance issue report template with new inputs#1680

Merged
AKuHAK merged 3 commits intops2homebrew:masterfrom
Jaajees:Update-issue-template
Apr 9, 2026
Merged

Enhance issue report template with new inputs#1680
AKuHAK merged 3 commits intops2homebrew:masterfrom
Jaajees:Update-issue-template

Conversation

@Jaajees
Copy link
Copy Markdown
Contributor

@Jaajees Jaajees commented Mar 12, 2026

Added input fields for extra features and compatibility modes in the issue report template.

Pull Request checklist

Note: these are not necessarily requirements

  • I reformatted the code with clang-format
  • I checked to make sure my submission worked
  • I am the author of submission or have permission from the original author
  • Requires update of the PS2SDK or other dependencies
  • Others (please specify below)

Pull Request description

Add more fields on template issue

  • Extra Features
  • Compatibility Modes

@J013k old suggestion

Jaajees added 2 commits March 12, 2026 09:44
Added input fields for extra features and compatibility modes in the issue report template.
Comment thread .github/ISSUE_TEMPLATE/issue-report.yml Outdated
placeholder: "eg: None, VMC, PADEMU, GSM..."

- type: input
id: compat-modes
Copy link
Copy Markdown
Member

@israpps israpps Mar 12, 2026

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Dropdowns they would limit the choices

I myself use GSM, PADEMU and VMC simultaneously in some games.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

it makes sure the user selects actual features, and it can be configured for both single or multi selection

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Perhaps checkboxes would be better in this context.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@israpps

Looks good to me.
Any other suggestions?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

What do you think @AKuHAK

…opdown

Fix YAML syntax error and replaced input for extra features with a dropdown
@AKuHAK AKuHAK requested a review from Copilot April 9, 2026 21:16
@AKuHAK AKuHAK merged commit 39afed2 into ps2homebrew:master Apr 9, 2026
18 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Enhances the GitHub issue report form by capturing additional configuration details that are commonly needed to triage OPL issues.

Changes:

  • Added a multi-select dropdown for enabled extra features (e.g., VMC/PADEMU/GSM).
  • Added an input field for reporting tested compatibility modes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- HDD
- iLink
- MX4SIO

Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

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

Line contains trailing whitespace on an otherwise blank line. This adds noise to diffs and can cause unnecessary merge conflicts; please remove the whitespace-only line (or make it a truly empty line).

Suggested change

Copilot uses AI. Check for mistakes.
Comment on lines +83 to +84
description: Which compatibility modes were tested.
placeholder: "eg: mode 6, 6+3, 1+3+6..."
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

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

Minor grammar/consistency: consider using “e.g.” instead of “eg” in the placeholder, and phrasing the description as a question (e.g., ending with a “?”) since other fields use question-style prompts.

Suggested change
description: Which compatibility modes were tested.
placeholder: "eg: mode 6, 6+3, 1+3+6..."
description: Which compatibility modes were tested?
placeholder: "e.g. mode 6, 6+3, 1+3+6..."

Copilot uses AI. Check for mistakes.
@J013k
Copy link
Copy Markdown
Contributor

J013k commented Apr 11, 2026

Is there a chance to also add these questions as mandatory?
There will not be a chance to create an issue without it.

I mean at least these fields:
https://i.imgur.com/bj4SXkT.png

Because when I see a report with almost all No response fields it is hard to figure it out with what we are dealing with:
https://i.imgur.com/cLlbgRe.png

Additionally most of these report are pointless, because if someone do not give a d*** to fill them,
in 99% it is an end user fault with an ISO or a device not OPL.

PS: I almost forgot, what about also add a field with Game ID e.g. What is your game ID? E.g. SLES_500.01.

@Jaajees
Copy link
Copy Markdown
Contributor Author

Jaajees commented Apr 11, 2026

@J013k , @AKuHAK

Now, all fields are required

The reference to the latest stable version or repository versions on MEGA are really necessary?
In the end we will always we will ask the user to test on the latest version.

@Jaajees Jaajees mentioned this pull request Apr 19, 2026
5 tasks
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