Skip to content

Bump docfx to 2.80.1 - #10

Merged
masa-iwm merged 1 commit into
mainfrom
claude/project-thread-cs0fli
Sep 25, 2026
Merged

masa-iwm merged 1 commit into
mainfrom
claude/project-thread-cs0fli

Conversation

@masa-iwm

Copy link
Copy Markdown
Owner

Requested by masa · project thread

What this changes

Before: docfx is pinned to 2.78.5. Dependabot's update to 2.80.1 (#7) also drops the final newline from .config/dotnet-tools.json, which .editorconfig (insert_final_newline = true) requires.

After: docfx is pinned to 2.80.1 and the file keeps its final newline. This replaces #7; Dependabot closes it once main carries 2.80.1.

How: a one-line version change in .config/dotnet-tools.json. PR CI does not run docfx (the Docs workflow runs on pushes to main only), so the site was built locally from main with 2.78.5 and with 2.80.1 and compared: both exit 0, the same 9 warnings, the same set of pages. 14 API pages change, all because code blocks inside XML comments are no longer wrapped in <p> (docfx fix dotnet/docfx#11143); the rest is the template's rebuilt JS bundles under public/.

Checklist

The long form of each gate is in
CONTRIBUTING.md.

  • dotnet build is warning-free. Not affected: only the docfx tool pin changes; CI covers the build.
  • The generator agrees with what is committed. Not affected; CI verify covers it.
  • Nothing under src/*/Generated/ was hand-edited.
  • Census counts were changed deliberately if the emitted surface moved. Not affected.
  • Nothing the published baseline shipped was removed or reshaped. Not affected.
  • dotnet test passes. Not run locally (no project code changes); left to the CI matrix.
  • NativeAOT smoke checked when the runtime or marshalling was touched. Not touched.
  • English throughout.
  • Commits are in the imperative mood, one logical change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01AxJrKQ3nzaSxVXcYzZkVu3


Generated by Claude Code

Dependabot's update (#7) drops the final newline from
.config/dotnet-tools.json, which .editorconfig requires, so the bump
is made here with the newline kept.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@masa-iwm masa-iwm self-assigned this Sep 25, 2026
@masa-iwm
masa-iwm marked this pull request as ready for review September 25, 2026 13:19
@masa-iwm
masa-iwm merged commit cf45c5f into main Sep 25, 2026
10 checks passed
@masa-iwm masa-iwm mentioned this pull request Sep 25, 2026
9 tasks done
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