Skip to content

Include wiki links in error output#200

Merged
philiptaron merged 2 commits intoNixOS:mainfrom
philiptaron:error-wiki-links
Feb 18, 2026
Merged

Include wiki links in error output#200
philiptaron merged 2 commits intoNixOS:mainfrom
philiptaron:error-wiki-links

Conversation

@philiptaron
Copy link
Copy Markdown
Contributor

@philiptaron philiptaron commented Feb 16, 2026

Each error from nixpkgs-vet now includes a link to its wiki documentation page. In terminal output, the NPV code is a clickable OSC 8 hyperlink. Colored output uses bold for the path, red for the error message, and default color for the link.

Errors that don't follow the - path: message pattern fall back to all-red with the link appended. Plain text output appends (https://github.com/NixOS/nixpkgs-vet/wiki/NPV-XXX) to each error

Completes #99

Test plan

  • All existing tests updated with wiki URLs in expected output
  • cargo test passes
  • cargo fmt -- --check passes

@philiptaron philiptaron requested a review from a team as a code owner February 16, 2026 22:38
@philiptaron philiptaron linked an issue Feb 16, 2026 that may be closed by this pull request
@philiptaron philiptaron reopened this Feb 16, 2026
@philiptaron philiptaron force-pushed the error-wiki-links branch 3 times, most recently from a9f5b88 to 389e3a6 Compare February 17, 2026 00:06
Each error now shows its wiki URL. In terminal output, the NPV code
is an OSC 8 clickable hyperlink; in plain text it's a parenthesized URL.

Part of NixOS#99
For errors following "- path: message", the path is now bold and the
message is red. The wiki link uses an OSC 8 hyperlink (clickable NPV
code) in terminal output. Errors with other formats fall back to
all-red with the link appended.
@philiptaron philiptaron merged commit 1a6cd11 into NixOS:main Feb 18, 2026
4 of 5 checks passed
@philiptaron philiptaron deleted the error-wiki-links branch February 18, 2026 21:49
@mdaniels5757 mdaniels5757 mentioned this pull request Mar 22, 2026
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.

Provide web documentation for each error

2 participants