Skip to content
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

Consider moving off of package:stager #8666

Open
mossmana opened this issue Jan 6, 2025 · 1 comment
Open

Consider moving off of package:stager #8666

mossmana opened this issue Jan 6, 2025 · 1 comment
Labels
blocked Issue or PR is blocked on another issue or PR. cost: medium Medium engineering cost to implement P3 issues we think are valid but not important tech debt

Comments

@mossmana
Copy link
Contributor

mossmana commented Jan 6, 2025

After pulling from the latest master of devtools, I received the following error:

Because devtools_app depends on integration_test from sdk which depends on test_api 0.7.4, test_api 0.7.4 is required.
And because test 1.25.8 depends on test_api 0.7.3, test 1.25.8 is forbidden.
And because devtools_tool depends on test ^1.25.8 and no versions of test match >1.25.8 <1.25.9, test ^1.25.9 is required.
And because test >=1.25.9 depends on analyzer >=6.0.0 <8.0.0 and stager 1.0.1 depends on analyzer ^5.1.0, stager 1.0.1 is forbidden.
Because no versions of stager match >1.0.1 <2.0.0 and devtools_app depends on stager ^1.0.1, stager 1.0.1 is required.
Thus, version solving failed.

The workaround is to add the following dependency_overrides to the top level pubspec.yaml:

dependency_overrides:
  analyzer: 6.11.0
  dart_style: 2.3.3

The longer term fix will require stager to update it's dependencies:
bryanoltman/stager#48

We should make a decision if we should continue to depend on stager going forward if the versioning issue cannot be addressed from the stager side.

@kenzieschmoll kenzieschmoll changed the title Dependency version match error Consider moving off of package:stager Jan 10, 2025
@kenzieschmoll kenzieschmoll added blocked Issue or PR is blocked on another issue or PR. P3 issues we think are valid but not important labels Jan 10, 2025
@kenzieschmoll
Copy link
Member

Once Widget Previews are available, we should be able to migrate off of the stager package and use Previews for the current set of scenes. CC @bkonyi this could be a good real-app use case for beta testing previews.

@kenzieschmoll kenzieschmoll added tech debt cost: medium Medium engineering cost to implement labels Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Issue or PR is blocked on another issue or PR. cost: medium Medium engineering cost to implement P3 issues we think are valid but not important tech debt
Projects
None yet
Development

No branches or pull requests

2 participants