Uh oh!
There was an error while loading. Please reload this page.
Add modern local library build script and process - #789
Merged
Conversation
Unified PowerShell script for local NuGet feed workflow using LOCAL_NUGET_REPO env var. Reads version from SilVersions.props, packs with symbols, copies PDBs, clears cache. Works for liblcm, libpalaso, and chorus. Updates nuget.config, build.ps1, and docs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Pack-LocalLibrary.ps1 now appends '-local' to the version and updates SilVersions.props so NuGet unambiguously resolves from the local feed. Also fix dotnet pack arg order and simplify nuget.config source mapping to use '*' on both sources. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use pipeline Copy-Item and wildcard Get-ChildItem paths to simplify PDB copying and cache clearing loops. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Support multiple libraries per invocation, detect packed version from output, and add set-version mode. Enforce libpalaso-first pack order. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use explicit -Palaso/-Lcm/-Chorus switches to opt in, with optional -PalasoPath/-LcmPath/-ChorusPath to override env vars. Default is no local work unless a switch is set. Update docs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The %LOCAL_NUGET_REPO% env var expansion fails in CI when unset, causing NU1301. Instead, Manage-LocalLibraries.ps1 now adds the local source to nuget.config dynamically when packing. Revert with: git checkout nuget.config Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jasonleenaylor
marked this pull request as ready for review
March 30, 2026 17:50
Script now uses dotnet nuget add source instead of modifying the repo's nuget.config. Source detection matches by path to avoid duplicates. Removed packageSourceMapping so user-level sources work without mapping conflicts. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
papeh
approved these changes
Mar 30, 2026
| .EXAMPLE | ||
| .\Build\Manage-LocalLibraries.ps1 -Palaso -Chorus -ChorusPath C:\Repos\chorus | ||
| Packs libpalaso (from env var) and chorus first, then chorus. |
Contributor
There was a problem hiding this comment.
Packs libpalaso (from env var) --and chorus-- first, then chorus.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove the old bash script and add a powershell script for local library development
Document the script and process for agents and developers
This change is