Skip to content

De-document styling of warnings text#1670

Merged
rolandwalker merged 1 commit intomainfrom
RW/de-document-warnings-text-styles
Mar 2, 2026
Merged

De-document styling of warnings text#1670
rolandwalker merged 1 commit intomainfrom
RW/de-document-warnings-text-styles

Conversation

@rolandwalker
Copy link
Contributor

Description

De-document styling of warnings text, by removing the relevant entries from myclirc and amending the changelog.

Motivation: there are some unexplored bugs in applying styling to warnings text elements, and it is better for main to stay releasable.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

by removing the relevant entries from myclirc and amending the
changelog.

Motivation: there are some unexplored bugs in applying styling to
warnings text elements, and it is better for main to stay releasable.
@rolandwalker rolandwalker self-assigned this Mar 2, 2026
@github-actions
Copy link

github-actions bot commented Mar 2, 2026

No blocking findings in the PR diff (e14c630c...7087c76).

  • Correctness/regressions: No functional code paths changed; only docs/sample config entries and changelog wording were updated.
  • Security: No security-impacting changes.
  • Tests/edge cases: No new test gaps introduced by this docs-only change.

Residual risk to track separately: warning-style keys are still present in runtime style mapping/code paths, just undocumented, so existing users with warnings.* overrides may still hit the underlying unresolved bugs.

@rolandwalker rolandwalker merged commit f47ec10 into main Mar 2, 2026
10 checks passed
@rolandwalker rolandwalker deleted the RW/de-document-warnings-text-styles branch March 2, 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