Repository navigation
docs: primary constructor tip and correct minimum SDK - #100
Merged
Merged
Conversation
Contributor
|
📖 Docs preview: https://kaisel.dev/pr-100/ (updates on every push; removed when this PR closes) |
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.
Docs only. Primary constructor guidance, plus a corrected minimum SDK.
Getting started. Adds a tip after the first route example showing the same routes written with Dart 3.13 primary constructors. The classic form stays the main example because kaisel's floor is Dart 3.10.
Concepts. A short tip in "A route is a value" pointing to the Getting started form.
kaisel skill. A new rule in "Defining routes": read the
environment: sdk:lower bound in the user's pubspec, use primary constructors at^3.13.0or higher and the classic form below it, never mix the two in one file, and never raise the user's SDK just for this.propsis still required for equality.Minimum versions. The page said "Flutter 3.24+ and requires Dart 3", but the packages declare
sdk: ^3.10.0, so the real floor is Dart 3.10 (Flutter 3.38).All snippets were checked in a scratch package on
sdk: ^3.13.0against kaisel_core: positional, named optional and no-field variants analyze clean, equality viapropsholds, and an exhaustiveswitchmatches them.