Clarify NodaTime serializer configuration - #135
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Coverage Report for CI Build 32188301679Warning No base build found for commit Coverage: 87.613%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
Summary
JsonSerializerOptionsandNodaTimeJobSerializerregistration as the primary customization pathIJobSerializeris preserved by Jobs registrationAddImmediateJobsNodaTime()as an optional defaults-only convenienceStack
This PR is stacked on #134 because the package-specific NodaTime README is introduced there. After #134 merges, this PR can be retargeted to
main.Validation
dotnet pack src/Immediate.Jobs.NodaTime/Immediate.Jobs.NodaTime.csproj -c Release -v minimal