Harden the release pipeline and document the generated surface - #6
Merged
Merged
Conversation
sbokatuk
commented
Jul 24, 2026
Owner
- README: fix the stale 3.14.0.1 install pins, and fail CI when README versions drift from Directory.Build.props (build/CheckReadmeVersions.sh)
- Pack the curated docs/release-notes/.md into PackageReleaseNotes, with the releases page as fallback, so nuget.org stops showing empty notes
- Document why DDLogger's error/attributes parameters carry no [NullAllowed]: upstream declares them _Nonnull and the Swift implementation takes a non-optional [String: Any], so letting null through would trade today's managed ArgumentNullException for a native crash. The README's troubleshooting section points at the null-tolerant Log(...) instead.
- Add doc summaries to the 26 main entry types in the ApiDefinitions files; the binding generator carries them into the packages' XML docs (measured, not assumed), so DDDatadog, DDConfiguration, the per-module Enable types, DDRUMMonitor, DDLogger and the header writers now have IntelliSense
- Turn the trim and AOT analyzers on permanently; the whole surface analyses clean today, and IsTrimmable stays deliberately unset - the props explain
- Add build/DiffSwiftHeaders.sh and docs/regenerating-bindings.md: the header diff is the upgrade work list while Objective Sharpie cannot regenerate, and the script also yields the SHA-256 checksums.txt records
- Document the net8 sunset: dropped in the first release after .NET 8 leaves support on 10 November 2026
- README: fix the stale 3.14.0.1 install pins, and fail CI when README versions drift from Directory.Build.props (build/CheckReadmeVersions.sh) - Pack the curated docs/release-notes/<version>.md into PackageReleaseNotes, with the releases page as fallback, so nuget.org stops showing empty notes - Document why DDLogger's error/attributes parameters carry no [NullAllowed]: upstream declares them _Nonnull and the Swift implementation takes a non-optional [String: Any], so letting null through would trade today's managed ArgumentNullException for a native crash. The README's troubleshooting section points at the null-tolerant Log(...) instead. - Add doc summaries to the 26 main entry types in the ApiDefinitions files; the binding generator carries them into the packages' XML docs (measured, not assumed), so DDDatadog, DDConfiguration, the per-module Enable types, DDRUMMonitor, DDLogger and the header writers now have IntelliSense - Turn the trim and AOT analyzers on permanently; the whole surface analyses clean today, and IsTrimmable stays deliberately unset - the props explain - Add build/DiffSwiftHeaders.sh and docs/regenerating-bindings.md: the header diff is the upgrade work list while Objective Sharpie cannot regenerate, and the script also yields the SHA-256 checksums.txt records - Document the net8 sunset: dropped in the first release after .NET 8 leaves support on 10 November 2026 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Version bump, README pins to match (now CI-enforced), and the curated release notes - which this release is the first to surface on nuget.org through PackageReleaseNotes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
DatadogNet.WebViewTracking.iOS joins the sample - the one package with a callable API that nothing exercised - with a tracked WKWebView page and a README usage section to match. A button makes an instrumented NSUrlSession request through the generic Enable<TDelegate>(), the automatic path the sample only ever showed manually, and another toggles a per-view Session Replay privacy override. Release notes extended accordingly. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This branch was previously deployed
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.