Update docs for Freezed v3#1183
Conversation
WalkthroughThe changes update the Freezed package’s documentation, configuration, and Dart code. The README was revised for clarity, removing outdated notes and adding alternative pattern matching options. In configuration files, several builder options were removed, and dependency versions were upgraded alongside an SDK constraint update. Code modifications include adding an override annotation, updating method signatures to accept optional parameters, converting an abstract union class to a sealed class with a switch expression for pattern matching, and changing certain classes to mixin classes. Changes
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (6)
💤 Files with no reviewable changes (1)
🚧 Files skipped from review as they are similar to previous changes (5)
⏰ Context from checks skipped due to timeout of 90000ms (3)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Also, is this the best way to bootstrap the package: If you'd accept a PR to the Melos v7 dev releases (which have seemed stable in my testing, and I think resolves your issues with Melos>3), I'd be happy to oblige. Same with Riverpod. |
70d9144 to
6ae2b13
Compare
|
Found some more outdated docs :) |
|
@rrousselGit, should freezed/example include freezed_lint, or should only the freezed_lint example include it? |
Note: `diagnosticable.dart` doesn't work right now, see rrousselGit#1184.
40b8261 to
10d052f
Compare
|
I don't care either way |
I use melos 2.0 on purpose. I don't want to upgrade |
|
@rrousselGit, can we merge this and leave the other changes for follow ups? |
|
Thanks! |
Hopefully this will help your reputation 😜
The changes in the example got the analyzer to shut up, haven't looked too closely to see if they're actually correct, especially around
mixin class.The
_ToStringclass indiagnosticable.dartdoesn't work, I think it's a bug. See #1184.Closes: #1181.
Summary by CodeRabbit
Summary by CodeRabbit