Repository navigation
build(aot): enable PublishTrimmed and PublishAot - #439
Conversation
Remove the Serilog.Enrichers.Demystifier package reference and its .Enrich.WithDemystifiedStackTraces() call in ServerStartup, ahead of publishing trimmed AOT builds. Refs #430
…mage (#430) Enable InvariantGlobalization in BuildBot.csproj and remove the en-US culture validation from ServerStartup.cs, which is no longer meaningful under invariant mode. Switch the Dockerfile to the plain runtime-deps 11.0 resolute chiseled image, as ICU is no longer required.
Enable PublishAot and PublishTrimmed in BuildBot.csproj and remove the -p:PublishAot=false override from src/publish so the publish script produces a trimmed native AOT build.
|
Development: implemented #430 in separate commits (head 8d6d3a0): buildcheck binding-generator property fix, removed System.Reactive/System.Interactive.Async, removed Serilog.Enrichers.Demystifier, InvariantGlobalization + plain chiseled runtime-deps image, PublishAot/PublishTrimmed, and the changelog entry. Local build, buildcheck, 391 tests and a linux-x64 AOT publish are clean; CI runs next. Not verified: AOT binary at runtime, and the linux-arm64 AOT compile (needs clang/zlib on the runner). Pre-existing, left alone: build-check action finds *.sln so misses src/BuildBot.slnx; src/publish has duplicate -p:FFPublishing/SolutionDir and conflicting IsProduction; Dockerfile has a stale commented FROM; dependabot.yml still lists Serilog.Enrichers.Demystifier; local dotnet test fails on SDK rc.3 vs rc.2 pinned in global.json. |
Roslyn analyzer findings
|
|
Simplify clean - advancing to code review |
|
Code review clean - advancing to security review |
|
Security review clean, advancing to coverage check |
|
Coverage ratchet passed - advancing to Human Review. .NET Overall 100.0% (baseline 100.0%); all 7 instrumented projects at 100.0%. COVERAGE.md left unchanged because the measured numbers are identical to main. |
|
Finalize: PR marked ready and auto-merge enabled (all required checks passed or skipped). |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Summary
Enable
PublishTrimmedandPublishAotfor the BuildBot publish configuration so the published output can be trimmed and compiled ahead-of-time.Closes #430