Complete Source Link config: add deterministic build, CI build, and explicit SourceRoot
- Dominant language
- C#
- Stars
- 307
- Forks
- 135
- Avg merge
- 1d 10h
- Merged PRs (30d)
- 4
Description
### Summary
This repository already references `Microsoft.SourceLink.GitHub` and sets `PublishRepositoryUrl` / `EmbedUntrackedSources`, so basic Source Link works. However, it is missing several elements of the canonical Source Link + deterministic-build configuration now standardized across the AWS .NET libraries (see aws/integrations-on-dotnet-aspire-for-aws#245):
- **`true`** — explicit (not relying on the SDK default).
- **CI-gated ``** — currently absent, so CI builds are not path-normalized/fully reproducible and build-agent paths can leak into the PDB.
- **Explicit ``** — needed because the AWS release pipeline packs from a source archive **without a `.git` directory**; without it Source Link cannot derive the repository root and emits no commit-pinned document map.
### Requested change
Bring the build configuration up to the template. Prefer consolidating into a shared `Directory.Build.props` (and `Directory.Packages.props` if using central package management) rather than duplicating across each `.csproj`:
```xml
git
true
true
true
true
```
### Why
Reproducible builds and reliable commit-pinned Source Link for the shipped NuGet packages, consistent with the rest of the AWS .NET library set.
### References
- Canonical template: aws/integrations-on-dotnet-aspire-for-aws#245
- Pipeline build-script revision support: aws/aws-dotnet-devex-pipelines#197
Contributor guide
Research direction
Start by locating the existing .csproj files and any shared Directory.Build.props or Directory.Packages.props, then review the current Microsoft.SourceLink.GitHub and repository metadata settings. Apply the requested deterministic-build, CI-gated, and SourceRoot configuration in the shared build files where appropriate. Done means CI and archive-based package builds produce reproducible output with commit-pinned Source Link mappings.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- build-system, ci-cd
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100