G-Research / G-Research/fsharp-analyzers
JsonSerializerOptionsAnalyzer should also mention .Default
Nobody has claimed this yet.
- Dominant language
- F#
- Stars
- 23
- Forks
- 3
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 6
Description
While I don't disagree with the advice in https://g-research.github.io/fsharp-analyzers/analyzers/JsonSerializerOptionsAnalyzer.html#JsonSerializerOptionsAnalyzer, I'd add a mention of JsonSerializerOptions.Default,
Going further, I believe the better overall pattern is using a Serdes wrapper with a mandatory options arg a la https://github.com/jet/FsCodec/blob/master/src/FsCodec.SystemTextJson/Serdes.fs#L7, but I can appreciate that shoehorning that advice/guidance into an Analyzer might be a stretch!
(Aside: this can be taken even futher by having a lazily-implemented .Default associated with each suite of Options a la
https://github.com/jet/FsCodec/blob/master/src/FsCodec.SystemTextJson/Options.fs#L14 and following the general pattern outlined in https://github.com/jet/fscodec?tab=readme-ov-file#examples-of-using-serdes-to-define-a-contract )
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the JsonSerializerOptionsAnalyzer guidance at the linked analyzer documentation page and review how its current advice is represented in the repository. Confirm where a reference to JsonSerializerOptions.Default belongs; done means the analyzer guidance mentions it without expanding into the optional Serdes design discussion.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fsharp
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100