Skip to content

Conversation

@tsavo-at-pieces
Copy link
Contributor

@tsavo-at-pieces tsavo-at-pieces commented Feb 10, 2026

Enhance cross-repo git workflows and AI integration

Branched from: main
Branch: dev
PR into: main

Summary

This PR introduces workspace dependency resolution for the dynamic_library package and its flutter_example application. This change simplifies dependency management within the monorepo, enabling packages to resolve local dependencies more effectively.

Changes

  • Add resolution: workspace to the pubspec.yaml file of the dynamic_library package.
  • Add resolution: workspace to the pubspec.yaml file within the flutter_example application.

Related PRs


This PR is part of a cross-repo change. All related PRs are listed above.

- Add `resolution: workspace` to `pubspec.yaml`
- Add `resolution: workspace` to `flutter_example/pubspec.yaml`
- Enable Dart/Flutter workspace functionality

Branch: dev
Base: main
Related to: cross-repo workspace change affecting 63 repositories
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds Dart/pub workspace dependency resolution configuration to this repo’s Dart package and Flutter example app to support monorepo-local dependency resolution.

Changes:

  • Added resolution: workspace to the root pubspec.yaml for dynamic_library.
  • Added resolution: workspace to flutter_example/pubspec.yaml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pubspec.yaml Opts the dynamic_library package into workspace resolution.
flutter_example/pubspec.yaml Opts the Flutter example app into workspace resolution.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

version: 1.0.0+1

environment:
sdk: ">=2.16.2 <3.0.0"
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

flutter_example declares environment.sdk: ">=2.16.2 <3.0.0", but it depends on the parent dynamic_library package (via path: ..), whose pubspec requires Dart >=3.9.0 <4.0.0. As a result, flutter pub get for the example app cannot succeed with the current SDK constraint. Update the example app's SDK constraint to be compatible with dynamic_library (and with resolution: workspace, if that feature requires a newer pub/Dart version).

Suggested change
sdk: ">=2.16.2 <3.0.0"
sdk: ">=3.9.0 <4.0.0"

Copilot uses AI. Check for mistakes.
Comment on lines 21 to 26
sdk: ">=2.16.2 <3.0.0"

resolution: workspace

# Dependencies specify other packages that your package needs in order to work.
# To automatically upgrade your package dependencies to the latest versions

This comment was marked as outdated.

- Remove `resolution: workspace` from `pubspec.yaml`
- Remove `resolution: workspace` from `flutter_example/pubspec.yaml`

Branch: dev
Base: main
Related to: cross-repo workspace change affecting 62 repositories
@tsavo-at-pieces
Copy link
Contributor Author

smart-prs is closing this PR automatically.

Reason: 0 file changes between dev and main (2 commit(s) ahead but identical content).

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.

1 participant