Skip to content

Conversation

@BalaSubramaniam12007
Copy link

@BalaSubramaniam12007 BalaSubramaniam12007 commented Jan 13, 2026

Fixes #478

Describe the changes you have made in this PR -
This PR resolves flutter pub get failed for new contributors following the readme instructions.

  1. SDK Constraint Update: Updated pubspec.yaml the environment to sdk: ">=3.9.0 <4.0.0".

This is required because shared_preferences ^2.5.4 (updated in #466) explicitly requires Dart ≥ 3.9.0.

  1. Documentation Update: Updated the README.md prerequisites to recommend Flutter ≥ 3.35.0 and Dart ≥ 3.9.0 to match the new project requirements.

  2. Onboarding Fix: Ensures new developers can successfully resolve dependencies and build the app without "forbidden version" errors during their initial local environment setup.

Screenshots of the changes (If any) -null

Note: Please check Allow edits from maintainers. If you would like us to assist with the PR.

Summary by CodeRabbit

  • Documentation

    • Updated prerequisites to reflect newer Flutter (3.35.0) and Dart (3.9.0) versions and clarified Android OAuth setup wording; minor formatting tweaks.
  • Chores

    • Raised the minimum Flutter/Dart SDK constraint in project configuration to require the newer SDK versions.

✏️ Tip: You can customize this high-level summary in your review settings.

@netlify
Copy link

netlify bot commented Jan 13, 2026

Deploy Preview for cv-mobile-app-web ready!

Name Link
🔨 Latest commit 2fba297
🔍 Latest deploy log https://app.netlify.com/projects/cv-mobile-app-web/deploys/6966751201ab420008474294
😎 Deploy Preview https://deploy-preview-479--cv-mobile-app-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link

coderabbitai bot commented Jan 13, 2026

Walkthrough

The README.md was edited to update the listed prerequisites from Flutter 3.32.2 (Dart 3.8.1) to Flutter 3.35.0 (Dart 3.9.0) and minor formatting around the Android OAuth Config text. The pubspec.yaml environment SDK constraint was changed from ">=3.7.2 <4.0.0" to ">=3.9.0 <4.0.0". No other dependency, public API, or behavioral changes were made.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: aligning SDK constraints in pubspec.yaml and updating README documentation with dependencies, matching the changeset.
Linked Issues check ✅ Passed All requirements from issue #478 are met: pubspec.yaml SDK constraint updated to >=3.9.0, README prerequisites updated to Flutter 3.35.0 and Dart 3.9.0, resolving the dependency conflict.
Out of Scope Changes check ✅ Passed All changes are directly scoped to resolving issue #478: SDK constraint alignment and README documentation updates with no unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9710e62 and 2fba297.

📒 Files selected for processing (1)
  • README.md
🔇 Additional comments (3)
README.md (3)

49-49: Version update is consistent with prerequisites.

The Android OAuth Config section correctly references Flutter 3.35.0, matching the updated prerequisites. The formatting change (added leading space) maintains consistent documentation style.


14-15: SDK constraint in pubspec.yaml is correctly aligned with README.

The pubspec.yaml file (not included in the review files) has the SDK constraint set to ">=3.9.0 <4.0.0", which correctly matches the Dart 3.9.0 documented in the README.


14-15: Flutter and Dart versions in README are correct. Flutter 3.35.0 ships with Dart 3.9.0, and shared_preferences ^2.5.4 requires minimum Dart ≥3.9.0. The prerequisite updates properly align with the dependency constraints.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
README.md (1)

49-49: Stale Flutter version reference.

Line 49 still references "Flutter 3.32.2" while the prerequisites now require Flutter 3.35.0. This inconsistency could confuse contributors.

📝 Suggested fix
-This project uses Flutter 3.32.2 and hence the support for compile-time variables. To use compile-time variables pass them in `--dart-defines` as `flutter run --dart-define=VAR_NAME=VAR_VALUE`. Supported `dart-defines` include :
+This project uses Flutter 3.35.0 and hence the support for compile-time variables. To use compile-time variables pass them in `--dart-defines` as `flutter run --dart-define=VAR_NAME=VAR_VALUE`. Supported `dart-defines` include :
🤖 Fix all issues with AI agents
In @README.md:
- Around line 14-15: In the README's Android OAuth Config section update the
outdated version token "Flutter 3.32.2" to "Flutter 3.35.0" so the prereqs match
the top-of-file Flutter/Dart versions; search for the literal string "Flutter
3.32.2" in the Android OAuth Config paragraph and replace it with "Flutter
3.35.0" (also fix the accidental "Flask" mention to "Flutter" if present).
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7317b6c and a3eff8f.

⛔ Files ignored due to path filters (1)
  • pubspec.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • README.md
  • pubspec.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Redirect rules - cv-mobile-app-web
  • GitHub Check: Header rules - cv-mobile-app-web
  • GitHub Check: Pages changed - cv-mobile-app-web
🔇 Additional comments (1)
pubspec.yaml (1)

17-18: LGTM! SDK constraint correctly aligned with dependency requirements.

The updated constraint >=3.9.0 <4.0.0 properly satisfies the shared_preferences ^2.5.4 requirement (line 54) and should resolve the dependency resolution failures for new contributors.

@BalaSubramaniam12007 BalaSubramaniam12007 changed the title Fix/sdk constraints chore(deps): align SDK constraints and README with dependencies Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inconsistent SDK constraints in pubspec.yaml vs shared_preferences ^2.5.4

1 participant