Skip to content

chore(.github): accept v3 bug reports, add triage dropdowns, wire up auto-labelers#5754

Merged
leaanthony merged 8 commits into
masterfrom
fix/issue-templates-triage
Jul 7, 2026
Merged

chore(.github): accept v3 bug reports, add triage dropdowns, wire up auto-labelers#5754
leaanthony merged 8 commits into
masterfrom
fix/issue-templates-triage

Conversation

@taliesin-ai

@taliesin-ai taliesin-ai commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

This PR refreshes the contributor-facing issue templates and triage automation for the v3 era.

Summary

  • bug_report.yml: removed the repeated "No bug reports are currently being accepted for Wails v3" banner — v3 bug reports are accepted. Added a required Wails version family dropdown (v3 / v2) and a required Operating System dropdown (Windows / macOS / Linux / iOS (experimental) / Android (experimental) / Multiple / Other). System Details now asks for wails3 doctor output (wails doctor for v2 projects). Fixed the v2-era links: troubleshooting now points at the stable v2 guide plus the v3 FAQ and feedback pages, and the TypeScript-generation note marks v2/internal/binding/binding_test as v2-only alongside the v3 generator path.
  • feature_request.yml: replaced the stale "No new feature requests are being accepted for Wails v1" notice with the discussion-first note (Discord / GitHub Discussions before opening an enhancement), and pointed substantial new functionality at the Wails Enhancement Proposal (WEP) process in v3/wep.
  • WEP documentation: the WEP process existed in v3/wep/README.md but was not referenced anywhere contributors actually look. It is now linked from the feature request template, CONTRIBUTING.md, and the v3 docs site: the contributing page gains a step-by-step "Propose an Enhancement (WEP)" summary and the feedback page's Suggestions tab points at it. Also fixed the stale feedback-guide link in CONTRIBUTING.md (/getting-started/feedback//feedback/).
  • The WEP PR is the single discussion venue: proposals are discussed on their PR (comments for feedback, thumbs-up reactions for support). Pre-proposal ideas can optionally be floated in the Ideas category on GitHub Discussions or on Discord. This keeps one canonical, greppable record per proposal instead of splitting the conversation across a PR and a discussion thread.
  • WEP process improvements: the README gains a "Do I Need a WEP?" section, a WEP Index (number, title, status, proposal, implementation) with documented status values, a standard decision-comment format, and an up-for-grabs rule for accepted proposals not started within 3 months. The template gains a metadata header (number, status, implementor) and a Platform Considerations section. The existing proposal is migrated as WEP 0001 (proposals/0001-titlebar-buttons/, status Implemented via [WEP] Customise Window Titlebars #3508).
  • documentation.yml: contribution-guidelines link now points at https://v3.wails.io/contributing/.
  • config.yml: blank_issues_enabled: false; Discord invite aligned with CONTRIBUTING.md's (discord.gg/JDdSxwjhGf).
  • Auto-labelers wired up: added .github/workflows/labeler.yml consuming the previously orphaned configs — github/issue-labeler@v3.4 for issues (.github/issue-labeler.yml, enable-versioned-regex: 0 to match the existing non-versioned regex format) and actions/labeler@v5 for PRs (.github/file-labeler.yml, converted in this PR to the v5 changed-files / any-glob-to-any-file format, same globs).
  • Deleted legacy Probot .github/stale.yml — superseded by .github/workflows/stale-issues.yml (actions/stale@v9).
  • Added CODE_OF_CONDUCT.md (Contributor Covenant v2.1): the beta-readiness audit flagged that the repo has no root community-health CoC file, even though the project has long published one at wails.io/coc. This adds the same Contributor Covenant at the repo root where GitHub surfaces it to contributors, with the same enforcement contact as the published page (coc@wails.io). The v3 docs contributing page now links to it.

Notes for the reviewer

  • All YAML validated with yaml.safe_load; issue forms checked against the GitHub issue-forms schema (body item types, dropdown options, required flags).
  • Please verify the repo's Actions permissions allow the new labeler workflow to run.

…templates

- bug_report.yml: remove the 'no v3 bug reports' banner (v3 bug reports
  are accepted), add required 'Wails version family' (v3/v2) and
  'Operating System' dropdowns, ask for wails3 doctor output (wails
  doctor for v2), fix v2-era links and mark the v2 binding tests
  reference as v2-specific alongside the v3 generator path
- feature_request.yml: replace the stale Wails v1 notice with the
  discussion-first note (Discord / GitHub Discussions), matching
  CONTRIBUTING.md
- documentation.yml: point the contribution guidelines link at
  https://v3.wails.io/contributing/
- config.yml: disable blank issues, align the Discord invite with
  CONTRIBUTING.md
- file-labeler.yml: convert to the actions/labeler v5 configuration
  format
- delete legacy Probot .github/stale.yml (superseded by
  workflows/stale-issues.yml using actions/stale@v9)
- add CODE_OF_CONDUCT.md (Contributor Covenant v2.1)
Consumes the previously orphaned .github/issue-labeler.yml
(github/issue-labeler@v3.4, non-versioned regex format) and
.github/file-labeler.yml (actions/labeler@v5) configs.
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

Walkthrough

This PR updates issue templates, labeler automation, contributor guidance, repository governance content, and WEP process documentation.

Changes

Issue templates and repo configuration

Layer / File(s) Summary
Bug report template guidance and doctor command
.github/ISSUE_TEMPLATE/bug_report.yml
Checklist wording and links are updated, and the requested doctor command changes to wails3 doctor with a v2 note.
Other issue templates
.github/ISSUE_TEMPLATE/config.yml, .github/ISSUE_TEMPLATE/documentation.yml, .github/ISSUE_TEMPLATE/feature_request.yml
Blank issues are disabled, the Discord contact link changes, the documentation contribution link changes, and feature request guidance is rewritten.
Labeler workflow and config
.github/file-labeler.yml, .github/workflows/labeler.yml
The file labeler config moves to the actions/labeler v5 schema, and a workflow is added to run issue and pull request labeling jobs.
Code of Conduct document
CODE_OF_CONDUCT.md
Adds Contributor Covenant v2.1 text with pledge, standards, scope, enforcement, and attribution sections.
Contributor and feedback guidance
CONTRIBUTING.md, docs/src/content/docs/contributing.mdx, docs/src/content/docs/feedback.mdx
Updates the v3 enhancement guidance, adds the WEP workflow, links to CODE_OF_CONDUCT.md, and adds WEP guidance to feedback docs.
WEP process and template updates
v3/wep/README.md, v3/wep/WEP_TEMPLATE.md, v3/wep/proposals/0001-titlebar-buttons/proposal.md
Expands the WEP process rules, updates the WEP template metadata and platform notes, and adds structured metadata to the implemented proposal example.

Estimated code review effort: 2 (Simple) | ~15 minutes

Possibly related PRs

  • wailsapp/wails#4566: Both PRs modify the bug report template's System Details/doctor command guidance.
  • wailsapp/wails#5269: Both PRs update guidance that distinguishes Wails v2 and v3 contributor flows.
  • wailsapp/wails#5271: Both PRs touch GitHub Actions labeling and issue-automation behavior.

Suggested labels: Documentation, size:L

Suggested reviewers: leaanthony

Poem

Hop, hop, I read the trail,
Templates tuned without a gale,
A code of conduct, clear and bright,
Labels whisked to match just right,
In docs and forms, the carrots glow,
The bunny nods: "Nice work!" 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The summary is present, but the required issue reference, type-of-change checklist, testing section, and checklist items are missing. Add the 'Fixes #...' line, complete the type-of-change and testing sections, and fill out the checklist items from the template.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title is concise and accurately reflects the PR's main changes to issue templates and auto-labeling.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/issue-templates-triage

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@taliesin-ai

Copy link
Copy Markdown
Collaborator Author

The labeler.yml workflow from the collapsed section below is now included in the PR itself (commit 11c6b64) — the workflow-scope push issue was resolved, so no separate maintainer step is needed. The PR is complete as-is: templates + config + labeler wiring.

The Wails Enhancement Proposal process (v3/wep) was not referenced
anywhere contributors actually look. Reference it from the feature
request template, CONTRIBUTING.md and the v3 docs site (contributing
page gets a step-by-step summary, feedback page a pointer).

Also fixes the stale feedback-guide link in CONTRIBUTING.md
(/getting-started/feedback/ -> /feedback/).
The v2 website has published a Contributor Covenant CoC at
wails.io/coc with coc@wails.io as the enforcement contact; use the
same address in the repo-level file.
Proposals are discussed in the Enhancement Proposals category on
GitHub Discussions. Discord remains available for informal chat but
is no longer the primary venue. Adds an explicit discuss-first step
to the WEP README's Idea Initiation section.
… WEP process

Implements the process improvements agreed with the maintainer:

- WEP Index table in the README (number, title, status, discussion,
  implementation) with documented status values
- Metadata header in the template (number, status, discussion link,
  implementor) and a Platform Considerations section
- 'Do I Need a WEP?' section defining when a proposal is required
- Support is now measured by upvotes on the Enhancement Proposals
  discussion rather than thumbs-ups on a draft PR
- Decisions recorded in a standard comment format; rejected proposals
  stay findable via the index
- Accepted proposals not started within 3 months become up for grabs
- Migrated the existing proposal as WEP 0001 (dir renamed to
  0001-titlebar-buttons, header added, status Implemented via #3508)

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@v3/wep/README.md`:
- Around line 56-60: The decision example in the README uses a bare fenced code
block, which triggers markdownlint MD040. Update the fenced snippet in the
decision example by adding a language tag such as text, keeping the content
unchanged, so the markdown example is properly labeled and lint-friendly.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2b8943d4-41fa-4dd9-9cb3-a623ab7ffc29

📥 Commits

Reviewing files that changed from the base of the PR and between e3f863b and 42d90fc.

📒 Files selected for processing (4)
  • docs/src/content/docs/contributing.mdx
  • v3/wep/README.md
  • v3/wep/WEP_TEMPLATE.md
  • v3/wep/proposals/0001-titlebar-buttons/proposal.md
✅ Files skipped from review due to trivial changes (2)
  • v3/wep/WEP_TEMPLATE.md
  • v3/wep/proposals/0001-titlebar-buttons/proposal.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/src/content/docs/contributing.mdx

Comment thread v3/wep/README.md
Comment on lines +56 to +60
```
**Decision**: Accepted / Rejected
**Decided by**: @maintainer(s)
**Rationale**: A short explanation of the decision.
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Label the decision example fence.

The bare fence here triggers MD040 and will keep markdownlint failing on this page. Use a language tag such as text for the snippet.

♻️ Proposed fix
-  ```
+  ```text
   **Decision**: Accepted / Rejected
   **Decided by**: `@maintainer`(s)
   **Rationale**: A short explanation of the decision.

</details>

<!-- suggestion_start -->

<details>
<summary>📝 Committable suggestion</summary>

> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

```suggestion

🧰 Tools
🪛 markdownlint-cli2 (0.22.1)

[warning] 56-56: Fenced code blocks should have a language specified

(MD040, fenced-code-language)

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@v3/wep/README.md` around lines 56 - 60, The decision example in the README
uses a bare fenced code block, which triggers markdownlint MD040. Update the
fenced snippet in the decision example by adding a language tag such as text,
keeping the content unchanged, so the markdown example is properly labeled and
lint-friendly.

Source: Linters/SAST tools

Per maintainer decision: proposals are discussed on the WEP PR itself
(comments for feedback, thumbs-up reactions for support), not in a
separate discussion thread. Pre-proposal ideas can optionally be
floated in the Ideas category on GitHub Discussions or on Discord.
Index column Discussion renamed to Proposal.
@leaanthony leaanthony merged commit 7d90315 into master Jul 7, 2026
12 of 17 checks passed
@leaanthony leaanthony deleted the fix/issue-templates-triage branch July 7, 2026 21:16
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