Skip to content

Fix packaging of design-time providers referenced via ProjectReference#19979

Open
T-Gro wants to merge 1 commit into
dotnet:mainfrom
T-Gro:fix/designtimeTP
Open

Fix packaging of design-time providers referenced via ProjectReference#19979
T-Gro wants to merge 1 commit into
dotnet:mainfrom
T-Gro:fix/designtimeTP

Conversation

@T-Gro

@T-Gro T-Gro commented Jun 22, 2026

Copy link
Copy Markdown
Member

Packing an F# library that activates a design-time type provider via a ProjectReference marked IsFSharpDesignTimeProvider="true" no longer included the provider assembly in the package (regressed by #18929).

dotnet pack collects content in an inner build with BuildProjectReferences=false, where @(_ResolvedProjectReferencePaths) is empty, so the AfterTargets="ResolveReferences" form of PackageFSharpDesignTimeTools produced nothing. It now resolves the provider with GetTargetPath — no reference build (safe under pack --no-build, no NETSDK1085) and no early ResolveReferences (so #18924 stays fixed). Registration moves back to evaluation time.

@github-actions

Copy link
Copy Markdown
Contributor

✅ No release notes required

@T-Gro T-Gro force-pushed the fix/designtimeTP branch 4 times, most recently from df6cb94 to 9e93c70 Compare June 22, 2026 11:21
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Tooling Safety Check — Affects-Build-Infra, Affects-Design-Time
Affects-Build-Infra: modifies Microsoft.FSharp.NetSdk.targets (MSBuild target execution)
Affects-Design-Time: changes design-time provider packaging/activation logic

Generated by PR Tooling Safety Check · opus46 5.9M ·

@github-actions github-actions Bot added ⚠️ Affects-Design-Time Tooling check: PR touches type providers or dependency manager ⚠️ Affects-Build-Infra Tooling check: PR touches build infrastructure labels Jun 22, 2026
@T-Gro T-Gro force-pushed the fix/designtimeTP branch 2 times, most recently from 7293dad to 4b67d9b Compare June 23, 2026 13:01
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@T-Gro T-Gro force-pushed the fix/designtimeTP branch from 4b67d9b to d3c65ef Compare June 23, 2026 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚠️ Affects-Build-Infra Tooling check: PR touches build infrastructure ⚠️ Affects-Design-Time Tooling check: PR touches type providers or dependency manager

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

1 participant