Skip to content

Fix docs workflow: resolve DocFX type resolution errors#49

Merged
jonathanvdc merged 2 commits intomasterfrom
copilot/fix-broken-docs-workflow
Mar 28, 2026
Merged

Fix docs workflow: resolve DocFX type resolution errors#49
jonathanvdc merged 2 commits intomasterfrom
copilot/fix-broken-docs-workflow

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 28, 2026

The docs workflow was failing with 120 MSBuild errors because docfx.json specified "TargetFramework": "net47" — a TFM that no longer exists in any project (projects target net10.0, netstandard2.0, or net472), and the Ubuntu runner has no .NET Framework reference assemblies.

Changes

  • docs/docfx.json: Change TargetFramework from net47net10.0 to match the actual project targets
  • docs/docfx.json: Add IL2LLVM/** to the metadata exclude list — it's an executable with cross-project references that don't belong in API docs

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

Copilot AI changed the title [WIP] Fix broken docs workflow to resolve type errors Fix docs workflow: resolve DocFX type resolution errors Mar 28, 2026
Copilot AI requested a review from jonathanvdc March 28, 2026 20:26
Copy link
Copy Markdown
Owner

@jonathanvdc jonathanvdc left a comment

Choose a reason for hiding this comment

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

Can you also exclude ILOpt in the same way that you excluded IL2LLVM?

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 28, 2026

Can you also exclude ILOpt in the same way that you excluded IL2LLVM?

ILOpt was already excluded in the original configuration — it's at line 14: "ILOpt/**". It uses the same glob pattern as the IL2LLVM/** entry I added, so no change is needed there.

Copilot AI requested a review from jonathanvdc March 28, 2026 20:36
@jonathanvdc jonathanvdc marked this pull request as ready for review March 28, 2026 20:38
@jonathanvdc jonathanvdc merged commit d80f3af into master Mar 28, 2026
2 checks passed
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