-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Bump mockito to 3.0.0-alpha+5 #17442
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
|
@jonahwilliams do you know why isolate deps would be disagreeing with themselves? I just ran |
|
@srawlins I'm not sure why specifically, but I do recall someone else having an issue with the isolate dep. |
|
apparently the analyzer had a dependency on isolate until recently |
|
Anything I can do or like... not? |
|
cc @devoncarew |
|
Hmm, |
|
Only two direct dependencies |
|
OK I think I found something. This bumps the version of test to The test package depends on the analyzer package. The latest analyzer, |
|
Opened a bug for @jonahwilliams or... whomever. #17473. |
|
@srawlins, I'm going to look at changing how we depend on |
|
@srawlins, you should be good to go. You'll have some merge conflicts :) You may want to re-apply the |
|
lol yeah its not easy recovering from these merges. I'll probably open a new PR in a bit. |
This is required to comply with the new noSuchMethod forwarding semantics coming from Dart soon. dart-lang/sdk#33031
It also bumps csslib *shwew*! And appears to remove need for barback and cli_util. Wins all around!