Skip to content

fix: the webkit2gtk4.0 should show webkit2gtk4.1#5115

Open
bitporo wants to merge 1 commit intowailsapp:v3-alphafrom
bitporo:v3-alpha
Open

fix: the webkit2gtk4.0 should show webkit2gtk4.1#5115
bitporo wants to merge 1 commit intowailsapp:v3-alphafrom
bitporo:v3-alpha

Conversation

@bitporo
Copy link
Copy Markdown

@bitporo bitporo commented Apr 4, 2026

I find the official website show the webkit2gtk version is 4.0, whether webkit2gtk4.1 should be displayed?

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 4, 2026

Walkthrough

Documentation update changing WebKitGTK dependency version from 4.0 to 4.1 across Linux installation instructions for Ubuntu/Debian and Fedora, including the wails3 doctor troubleshooting example.

Changes

Cohort / File(s) Summary
Linux Installation Documentation
docs/src/content/docs/quick-start/installation.mdx
Updated WebKitGTK package names from 4.0 to 4.1 for Ubuntu/Debian (libwebkit2gtk-4.0-devlibwebkit2gtk-4.1-dev) and Fedora (webkit2gtk4.0-develwebkit2gtk4.1-devel). Updated wails3 doctor troubleshooting reference to match.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • leaanthony

Poem

🐰 A whisker-twitch and a version bump,
WebKit climbs from four point oh,
Four point one, what a jump!
Linux docs now flow,
Dependencies dance just so! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is vague and lacks required information from the template, such as issue reference, change type selection, testing details, and checklist completion. Provide a clear issue reference (Fixes #), select the type of change (bug fix/feature/etc.), describe testing performed, and complete the required checklists in the PR description.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating documentation to reflect WebKit2GTK 4.1 instead of 4.0 across Linux installation instructions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
docs/src/content/docs/quick-start/installation.mdx (1)

331-335: Clarify this is an Ubuntu/Debian-specific example.

This Linux troubleshooting block uses apt; consider labeling it explicitly as “Ubuntu/Debian example” to avoid confusion for Fedora/Arch users.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/src/content/docs/quick-start/installation.mdx` around lines 331 - 335,
Update the Linux troubleshooting paragraph that shows an apt install example so
it explicitly notes this is an Ubuntu/Debian-specific example: modify the
"**Linux:**" block (the code block showing "❌ webkit2gtk not found" and "Install
with: sudo apt install libwebkit2gtk-4.1-dev") to prepend or append a short
qualifier such as "Ubuntu/Debian example" or "Debian/Ubuntu (uses apt)" and
optionally add a brief hint for other distributions (e.g., Fedora/Arch users use
dnf/pacman equivalents). Ensure the wording is concise and keeps the existing
example intact while clearly labeling it as distro-specific.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@docs/src/content/docs/quick-start/installation.mdx`:
- Around line 331-335: Update the Linux troubleshooting paragraph that shows an
apt install example so it explicitly notes this is an Ubuntu/Debian-specific
example: modify the "**Linux:**" block (the code block showing "❌ webkit2gtk not
found" and "Install with: sudo apt install libwebkit2gtk-4.1-dev") to prepend or
append a short qualifier such as "Ubuntu/Debian example" or "Debian/Ubuntu (uses
apt)" and optionally add a brief hint for other distributions (e.g., Fedora/Arch
users use dnf/pacman equivalents). Ensure the wording is concise and keeps the
existing example intact while clearly labeling it as distro-specific.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 920a2d04-8498-461b-987f-d885a2442802

📥 Commits

Reviewing files that changed from the base of the PR and between bb4fbf9 and 4b6d587.

📒 Files selected for processing (1)
  • docs/src/content/docs/quick-start/installation.mdx

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.

1 participant