chore(ci): bump Dart SDK pin to 3.13.4 - #140
Johnosezele wants to merge 2 commits into
Conversation
Pin every setup-dart step in CI and upstream-compat to 3.13.4, and note that pin in CONTRIBUTING.md. Package SDK constraints stay at ^3.10.0 and ^3.10.1. Exclude bdk_demo build, android, and ios from the analyzer so flutter pub get and flutter analyze match current Flutter stable.
|
Two questions: (1) The description mentions justfile changes (just deps, just format-check, just ci), but they're not in the diff. Did a commit not get pushed? (2) What was failing that needed the android/** / ios/** / build/** analyzer excludes? |
so i'm on dart 3.13.4 on my local, so when I run the format cmd's, certain files are reformatted because this new dart version has some updates on how dart format worked https://dart.dev/effective-dart/style#ordering, I think if you bump your local dart version to the latest, then format, you'd see some updates
So nothing failing, flutter/flutter#187940 Flutter 3.47's AnalysisOptionsMigration rewrites analysis_options.yaml during flutter pub get and flutter analyze unless these excludes are already present. bdk_demo only has android/, ios/, and build/, so those are the three entries it inserts.... flutter/flutter#187728 |
|
Thanks, that explains the test reformatting and why you added the analyzer exclusions. My first question was about the Justfile: the PR description says Since Dart 3.10 and 3.13 produce different formatting here, it would also help to say that contributors need Dart 3.13.4 for the format check. |
|
I've formated the description appropriately, thanks! |
|
The SDK bump and formatter changes look good. One thing still feels inconsistent: CONTRIBUTING says "Dart 3.10 or newer", but this PR shows that 3.10 and 3.13 can produce different formatting while CI is pinned to 3.13.4. Could we recommend Dart 3.13.4 specifically for contributors/formatting, while keeping the package SDK constraint at ^3.10.0? |
Summary
setup-dartstep inci.ymlandupstream-compat.ymlto Dart 3.13.4 (was 3.10.0), and note that pin inCONTRIBUTING.md. As per contributor requirement.^3.10.0bdk_demobuild/**,android/**, andios/**from the analyzer soflutter pub get/flutter analyzematch current Flutter stable.Test
build-and-test, smokes,upstream-compat)just cipasses locally on Dart 3.13.xflutter analyzeinbdk_demodoes not walkandroid//ios//build/