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

[Scheduled] flutter pub upgrade --null-safety #634

Closed
wants to merge 1 commit into from

Conversation

pubspec.yaml Outdated
@@ -61,7 +61,7 @@ dev_dependencies:
sdk: flutter
integration_test:
sdk: flutter
msix: ^3.6.2
Copy link
Owner Author

Choose a reason for hiding this comment

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

wow...

Copy link
Owner Author

Choose a reason for hiding this comment

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

Because msix >=3.0.0-beta.1 depends on cli_dialog ^0.5.0 which depends on dart_console ^1.0.0, msix >=3.0.0-beta.1 requires dart_console ^1.0.0.
And because dart_console >=1.0.0 depends on ffi ^1.0.0, msix >=3.0.0-beta.1 requires ffi ^1.0.0.
And because libedax4dart 5.3.1 depends on ffi ^2.0.1 and no versions of libedax4dart match >5.3.1 <6.0.0, msix >=3.0.0-beta.1 is incompatible with libedax4dart ^5.3.1.
So, because pedax depends on both libedax4dart ^5.3.1 and msix ^3.6.2, version solving failed.
pub get failed (1; So, because pedax depends on both libedax4dart ^5.3.1 and msix ^3.6.2, version solving failed.)

@sensuikan1973 sensuikan1973 self-assigned this Jun 22, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 22, 2022

Codecov Report

Merging #634 (c14d2d4) into main (133e18a) will decrease coverage by 0.30%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #634      +/-   ##
==========================================
- Coverage   95.53%   95.23%   -0.31%     
==========================================
  Files          48       48              
  Lines        1322     1322              
==========================================
- Hits         1263     1259       -4     
- Misses         59       63       +4     
Flag Coverage Δ
integration_test 74.35% <ø> (ø)
macos-latest 94.09% <ø> (-0.31%) ⬇️
ubuntu-latest 94.32% <ø> (+0.07%) ⬆️
widget_test 95.49% <ø> (-0.39%) ⬇️
windows-latest 94.17% <ø> (-0.38%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
lib/engine/api/count_bestpath.dart 100.00% <ø> (ø)
lib/engine/edax_server.dart 83.87% <0.00%> (-4.31%) ⬇️

@sensuikan1973
Copy link
Owner Author

convert Draft. See: #634 (comment)

@sensuikan1973 sensuikan1973 marked this pull request as draft June 22, 2022 21:20
@sensuikan1973 sensuikan1973 force-pushed the cron_upgrade_flutter_dependencies branch 7 times, most recently from 54c2bcb to d747cf0 Compare June 29, 2022 01:35
@sensuikan1973 sensuikan1973 force-pushed the cron_upgrade_flutter_dependencies branch 7 times, most recently from 8668d1c to 6cbc3ee Compare July 6, 2022 01:36
@sensuikan1973 sensuikan1973 force-pushed the cron_upgrade_flutter_dependencies branch 3 times, most recently from e134161 to 342f993 Compare July 9, 2022 01:37
@sensuikan1973 sensuikan1973 force-pushed the cron_upgrade_flutter_dependencies branch 6 times, most recently from 9912b48 to 694e551 Compare July 15, 2022 01:35
@sensuikan1973 sensuikan1973 force-pushed the cron_upgrade_flutter_dependencies branch from 694e551 to c25dcc0 Compare July 16, 2022 01:32
@sensuikan1973
Copy link
Owner Author

@sensuikan1973 sensuikan1973 deleted the cron_upgrade_flutter_dependencies branch July 16, 2022 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants