-
Notifications
You must be signed in to change notification settings - Fork 0
Bump SDK constraints and unify workspace dependencies #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Bump SDK constraint in `pubspec.yaml` to `>=3.9.0 <4.0.0` - Bump SDK constraint in `example/pubspec.yaml` to `>=3.9.0 <4.0.0` Branch: dev Base: main Related to: cross-repo workspace change affecting 63 repositories
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR aims to enable unified dependency resolution across the repository by opting the root package and the example app into Pub “workspace” resolution, alongside updating/confirming Dart SDK constraints.
Changes:
- Add
resolution: workspaceto the rootpubspec.yaml. - Add
resolution: workspacetoexample/pubspec.yaml. - (Per description) Set/confirm SDK constraints to
>=3.9.0 <4.0.0for both packages.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
pubspec.yaml |
Opts the root package into workspace dependency resolution. |
example/pubspec.yaml |
Opts the example app into workspace dependency resolution. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
pubspec.yaml
Outdated
| resolution: workspace | ||
|
|
Copilot
AI
Feb 10, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolution: workspace opts this package into Pub workspace resolution, but this repo doesn’t define a workspace root (no workspace: section found anywhere). As-is, dart pub get/flutter pub get is likely to fail or not behave as intended. Add a workspace: section at the workspace root listing members (e.g., include example/), or drop resolution: workspace if this repo isn’t meant to be a Pub workspace.
| resolution: workspace |
example/pubspec.yaml
Outdated
| resolution: workspace | ||
|
|
Copilot
AI
Feb 10, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolution: workspace requires this package to be part of a Pub workspace. The repo currently has no workspace: definition, so dependency resolution may fail or be inconsistent. Either add a workspace definition at the root (including example/) or remove resolution: workspace here.
| resolution: workspace |
- Remove `environment.sdk` constraint from `pubspec.yaml`. - Remove `environment.sdk` constraint from `example/pubspec.yaml`. Branch: dev Base: main Related to: cross-repo workspace change affecting 62 repositories
|
smart-prs is closing this PR automatically. Reason: 0 file changes between |
Summary
This PR updates the SDK constraints for the
code_text_fieldpackage and its example application to>=3.9.0 <4.0.0. It also introduces unified workspace dependency resolution to streamline development and ensure consistency across the project.Changes
>=3.9.0 <4.0.0inpubspec.yaml.resolution: workspaceto the rootpubspec.yaml.>=3.9.0 <4.0.0inexample/pubspec.yaml.resolution: workspacetoexample/pubspec.yaml.