Skip to content

docs: record Pygments 2.20 exclusion ADR#548

Merged
DecisionNerd merged 1 commit into
mainfrom
docs/547-pygments-220-adr
May 12, 2026
Merged

docs: record Pygments 2.20 exclusion ADR#548
DecisionNerd merged 1 commit into
mainfrom
docs/547-pygments-220-adr

Conversation

@DecisionNerd
Copy link
Copy Markdown
Owner

@DecisionNerd DecisionNerd commented May 12, 2026

Closes #547

Summary

  • Add ADR 0001 documenting why GraphForge intentionally excludes Pygments 2.20.x from docs dependencies.
  • Link the ADR from the Development docs navigation.
  • Capture the revisit criteria before relaxing the Pygments upper bound again.

Validation

  • make pre-push
  • uv run --group docs mkdocs build --strict

View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

- Add ADR documenting why docs dependencies exclude Pygments 2.20.x
- Link the ADR from the Development docs navigation

Validation: make pre-push
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 12, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ba1fb47f-fe53-45b5-beb9-9a440a3928d1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/547-pygments-220-adr

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

@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.81%. Comparing base (caaffe2) to head (815177d).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #548   +/-   ##
=======================================
  Coverage   83.81%   83.81%           
=======================================
  Files          47       47           
  Lines       12647    12647           
  Branches     3587     3587           
=======================================
  Hits        10600    10600           
  Misses       1265     1265           
  Partials      782      782           
Flag Coverage Δ
full-coverage 83.81% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
parser 92.93% <ø> (ø)
planner 79.90% <ø> (ø)
executor 75.46% <ø> (ø)
storage 98.68% <ø> (ø)
ast 97.51% <ø> (ø)
types 90.66% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update caaffe2...815177d. Read the comment docs.

@DecisionNerd DecisionNerd marked this pull request as ready for review May 12, 2026 06:04
@DecisionNerd DecisionNerd merged commit 8381da2 into main May 12, 2026
36 checks passed
@DecisionNerd DecisionNerd deleted the docs/547-pygments-220-adr branch May 12, 2026 06:05
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.

docs: record Pygments 2.20 exclusion decision

1 participant