Skip to content

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Dec 5, 2025

Description

Refines the issue explanations (flaws), in particular including the macro name.

Motivation

Make them clearer and add missing context, as they show up in the PR Review Companion comment in content PRs.

Additional details

# Regular issues.

-{url} is ill cased
+Link {url} is ill cased

-{url} is a redirect
+Link {url} is a redirect

-Can't resolve {url}
+Link {url} doesn't resolve

# Macro issues.

-Can't resolve {url}
+Macro {macroName} produces link {url} which doesn't resolve

-Macro produces link {url} which is a redirect
+Macro {macroName} produces link {url} which is a redirect

-{url} is ill cased
+Macro {} produces link {} which is ill cased

-Argument ({}) is not valid.
+Macro {macroName} received argument ({argument}) which is not valid

Related issues and pull requests

@caugner caugner requested a review from a team as a code owner December 5, 2025 20:34
@caugner caugner requested a review from argl December 5, 2025 20:34
In particular, mention macro name in flaw.
@caugner caugner force-pushed the refine-issue-explanations branch from 5635f4c to 75a9eb7 Compare December 5, 2025 20:39
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