Repository navigation
Enable interactive forms by default, inverting and renaming the previous experiment flag - #6107
Conversation
There was a problem hiding this comment.
Code Review
This pull request promotes the experimental interactive forms feature to a stable configuration setting named 'dart.interactiveForms' and enables it by default. However, in 'src/extension/analysis/analyzer.ts', the activation logic was incorrectly inverted to 'if (!config.interactiveForms)'. This bug should be resolved by removing the negation operator so that the feature is registered when enabled.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6107 +/- ##
==========================================
+ Coverage 71.13% 71.15% +0.02%
==========================================
Files 170 170
Lines 13079 13079
Branches 2626 2626
==========================================
+ Hits 9304 9307 +3
+ Misses 3265 3262 -3
Partials 510 510 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…ous experiment flag Rather than just deleting the flag, keep it around inverted for a little while so if any issues crop up, users can easily disable it (either to avoid them, or to confirm whether it's related to interactive forms).
a23b800 to
0414fbd
Compare
Rather than just deleting the flag, keep it around inverted for a little while so if any issues crop up, users can easily disable it (either to avoid them, or to confirm whether it's related to interactive forms).