Skip to content

fix: publish workflow — skip net10.0-only samples, add net10.0 SDK, pack both NuGet packages#14

Merged
elbruno merged 3 commits intomainfrom
fix/publish-workflow-net10-compat
Apr 4, 2026
Merged

fix: publish workflow — skip net10.0-only samples, add net10.0 SDK, pack both NuGet packages#14
elbruno merged 3 commits intomainfrom
fix/publish-workflow-net10-compat

Conversation

@elbruno
Copy link
Copy Markdown
Owner

@elbruno elbruno commented Apr 4, 2026

Problem

The \publish.yml\ workflow restored/built the entire solution with -p:TargetFrameworks=net8.0, but \src/samples/ZeroCloudRag/\ targets only
et10.0\ and depends on \ElBruno.LocalEmbeddings 1.0.1\ (net10.0-only), causing:

\
error NU1202: Package ElBruno.LocalEmbeddings 1.0.1 is not compatible with net8.0
\\

Additionally, the workflow only installed .NET 8.0 SDK and forced -p:TargetFrameworks=net8.0\ on the pack step, meaning the NuGet package only shipped the net8.0 TFM — missing net10.0.

Fix

  1. Setup .NET: Install both \8.0.x\ and \10.0.x\ SDKs
  2. Restore/Build: Target specific projects (libraries + tests) instead of the whole solution — avoids net10.0-only samples
  3. Tests: Run both \ElBruno.LocalLLMs.Tests\ and \ElBruno.LocalLLMs.Rag.Tests\
  4. Pack: Pack both \ElBruno.LocalLLMs\ and \ElBruno.LocalLLMs.Rag\ NuGet packages, without forcing -p:TargetFrameworks=net8.0\ so both TFMs are included
  5. Push: \�rtifacts/*.nupkg\ glob now picks up both packages

Failed run: https://github.com/elbruno/ElBruno.LocalLLMs/actions/runs/23982332047

elbruno and others added 3 commits April 4, 2026 11:57
- Orchestration logs: Trinity (XML docs), Tank (RAG tests)
- Session log: Squad release cycle (v0.11.0)
- Decision inbox: Merged tank-rag-tests.md to decisions.md
- Cross-agent updates: Trinity & Tank history.md appended

Release: v0.11.0 (LocalLLMs 0.11.0, Rag 0.2.0)
Issues #11, #12 closed
No regressions, full test coverage achieved

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ack both NuGet packages

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@elbruno elbruno merged commit c03947c into main Apr 4, 2026
1 of 3 checks passed
@elbruno elbruno deleted the fix/publish-workflow-net10-compat branch April 4, 2026 16:03
elbruno added a commit that referenced this pull request Apr 4, 2026
Same fix as publish.yml (PR #14): restore/build only library + test
projects to avoid net10.0-only sample incompatibility.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.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.

1 participant