Skip to content

Fix DocFX configuration and add documentation content#73

Merged
Chris-Wolfgang merged 4 commits into
mainfrom
docs/fix-docfx-configuration
Apr 16, 2026
Merged

Fix DocFX configuration and add documentation content#73
Chris-Wolfgang merged 4 commits into
mainfrom
docs/fix-docfx-configuration

Conversation

@Chris-Wolfgang
Copy link
Copy Markdown
Owner

Summary

  • Fix blank docs site by replacing all repo-template placeholders in docfx_project/ with real values
  • Write actual documentation content: introduction, getting started guide, API reference landing page
  • Fix docfx.json project path (src/<path>/<project>.csprojsrc/**/*.csproj), app metadata, and template config
  • Fix docs/toc.yml project website URL (was pointing to DbContextBuilder)
  • Add logo.svg, root toc.yml, api/index.md, and .gitignore files for generated output

Verified

  • docfx metadata + docfx build pass with 0 warnings, 0 errors
  • Generated site has landing page, documentation articles, and API reference for Try, Result, Result<T>

Test plan

🤖 Generated with Claude Code

The docs site was blank because docfx_project/ still contained
repo-template placeholders. Replace all placeholders with real values
and write actual documentation content.

- docfx.json: fix project path, app name, title, base URL, templates
- Add root toc.yml linking Documentation and API Reference sections
- Add logo.svg (branded W mark)
- Write landing page (index.md) with quick links and install snippet
- Write introduction.md covering Try, Result, Result<T>, and composition
- Write getting-started.md with installation and usage examples
- Fix docs/toc.yml: correct project URL (was DbContextBuilder)
- Add api/index.md for API reference landing page
- Add .gitignore for api/ (generated yml) and _site/ (build output)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 16, 2026 01:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the DocFX documentation project to produce a non-blank GitHub Pages site for the Try-Pattern library, replacing template placeholders and adding real documentation content plus supporting assets/config.

Changes:

  • Fix DocFX metadata/build configuration (project globbing, templates, site metadata/assets).
  • Add documentation content (landing page, introduction, getting started, API landing page) and TOCs.
  • Add supporting assets and gitignores for generated API/build outputs.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
docfx_project/docfx.json Updates DocFX metadata/build config, templates, and site metadata.
docfx_project/index.md Replaces placeholder landing page with real project overview and links.
docfx_project/toc.yml Adds top-level DocFX TOC linking docs + API sections.
docfx_project/docs/toc.yml Fixes docs section TOC casing and project website URL.
docfx_project/docs/index.md Adds docs section landing page.
docfx_project/docs/introduction.md Adds intro content and examples for core types.
docfx_project/docs/getting-started.md Adds install/usage guide and examples.
docfx_project/api/index.md Adds API section landing page.
docfx_project/api/.gitignore Ignores generated API metadata YAML files.
docfx_project/_site/.gitignore Ignores generated DocFX build output.
docfx_project/logo.svg Adds a logo asset for site branding.

Comment thread docfx_project/docfx.json Outdated
Comment thread docfx_project/index.md
Comment thread docfx_project/docs/getting-started.md Outdated
Comment thread docfx_project/docs/getting-started.md Outdated
Chris-Wolfgang and others added 2 commits April 15, 2026 21:41
- Remove _baseUrl from docfx.json (conflicts with versioned deployment
  under /versions/<tag>/)
- Change API links from .yml to api/index.md (yml files aren't served)
- Update prerequisites to .NET 10 SDK (repo multi-targets net10.0)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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