Skip to content

Update issue templates - #40

Merged
vinpogo merged 2 commits into
mainfrom
vinpogo-patch-1
Mar 3, 2026
Merged

Update issue templates#40
vinpogo merged 2 commits into
mainfrom
vinpogo-patch-1

Conversation

@vinpogo

@vinpogo vinpogo commented Mar 3, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 3, 2026 07:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds GitHub issue templates to standardize how users file bugs and request features in the repository.

Changes:

  • Added a Markdown-based Feature request issue template.
  • Added a Markdown-based Bug report issue template with environment capture prompts.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
.github/ISSUE_TEMPLATE/feature_request.md Introduces a feature request template with problem/solution/alternatives/context prompts.
.github/ISSUE_TEMPLATE/bug_report.md Introduces a bug report template with reproduction steps, expected behavior, and environment details.

Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
Comment on lines +28 to +35
- Browser [e.g. chrome, safari]
- Version [e.g. 22]

**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]

Copilot AI Mar 3, 2026

Copy link

Choose a reason for hiding this comment

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

The "Browser" bullet in the Smartphone section is missing a colon after the field name, unlike the other bullets in the section. Adding the colon will keep the fields consistent.

Suggested change
- Browser [e.g. chrome, safari]
- Version [e.g. 22]
**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]
- Browser: [e.g. chrome, safari]
- Version: [e.g. 22]
**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser: [e.g. stock browser, safari]
- Version: [e.g. 22]

Copilot uses AI. Check for mistakes.
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
If applicable, add screenshots to help explain your problem.

**Desktop (please complete the following information):**
- OS: [e.g. iOS]

Copilot AI Mar 3, 2026

Copy link

Choose a reason for hiding this comment

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

In the Desktop section, the OS example is set to "iOS", which is a mobile OS and conflicts with the section’s intent. Consider using a desktop OS example (e.g., Windows/macOS/Linux) to avoid confusing reporters.

Suggested change
- OS: [e.g. iOS]
- OS: [e.g. macOS, Windows, Linux]

Copilot uses AI. Check for mistakes.
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
Comment on lines +28 to +35
- Browser [e.g. chrome, safari]
- Version [e.g. 22]

**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]

Copilot AI Mar 3, 2026

Copy link

Choose a reason for hiding this comment

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

The "Browser" bullet is missing a colon after the field name, unlike the surrounding bullets (OS/Version). Adding the colon will make the template more consistent and easier to fill out.

Suggested change
- Browser [e.g. chrome, safari]
- Version [e.g. 22]
**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]
- Browser: [e.g. chrome, safari]
- Version: [e.g. 22]
**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser: [e.g. stock browser, safari]
- Version: [e.g. 22]

Copilot uses AI. Check for mistakes.
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
- Version [e.g. 22]

**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]

Copilot AI Mar 3, 2026

Copy link

Choose a reason for hiding this comment

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

Minor formatting: "iPhone6" should include a space (e.g., "iPhone 6") to match typical device naming and improve readability in reports.

Suggested change
- Device: [e.g. iPhone6]
- Device: [e.g. iPhone 6]

Copilot uses AI. Check for mistakes.
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated

**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]

Copilot AI Mar 3, 2026

Copy link

Choose a reason for hiding this comment

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

Minor formatting: "iOS8.1" should include a space ("iOS 8.1") for readability and consistency with standard OS version formatting.

Suggested change
- OS: [e.g. iOS8.1]
- OS: [e.g. iOS 8.1]

Copilot uses AI. Check for mistakes.
@vinpogo
vinpogo force-pushed the vinpogo-patch-1 branch 2 times, most recently from 40bb7f6 to 5525827 Compare March 3, 2026 07:46
@vinpogo
vinpogo merged commit fc84f6f into main Mar 3, 2026
4 checks passed
@vinpogo
vinpogo deleted the vinpogo-patch-1 branch March 3, 2026 07:55
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.

2 participants