Skip to content
This repository was archived by the owner on May 25, 2026. It is now read-only.

Expand test_api dep to allow test_api 0.3.x - #353

Merged
srawlins merged 1 commit into
dart-archive:four-one-onefrom
srawlins:bump-test-api
Mar 11, 2021
Merged

srawlins merged 1 commit into
dart-archive:four-one-onefrom
srawlins:bump-test-api

Conversation

@srawlins

@srawlins srawlins commented Mar 10, 2021 •

Copy link
Copy Markdown
Contributor

This is a patch release for mockito 4.1.1.

Fixes dart-lang/build#4627

@google-cla google-cla Bot added the cla: yes label Mar 10, 2021
@srawlins
srawlins requested a review from natebosch March 10, 2021 00:46
@srawlins

Copy link
Copy Markdown
Contributor Author

CC @jonahwilliams

@jonahwilliams jonahwilliams left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Thank you!

Comment thread CHANGELOG.md
Comment thread pubspec.yaml
@zanderso

Copy link
Copy Markdown

This PR is needed to resolve a P1 for Flutter, but it looks like it might be stuck. What is the path forward here?

@srawlins

Copy link
Copy Markdown
Contributor Author

@natebosch is the versioning policy for technical reasons or just idiomatic? I don't know another version number that would be good for this release.

@natebosch

Copy link
Copy Markdown
Contributor

This PR is needed to resolve a P1 for Flutter

What is the P1 and what other paths to resolution were considered?

@zanderso

Copy link
Copy Markdown

This PR is needed to resolve a P1 for Flutter

What is the P1 and what other paths to resolution were considered?

Please see the linked issues flutter/flutter#77856 and dart-lang/build#4627

@srawlins
srawlins merged commit b1edbbb into dart-archive:four-one-one Mar 11, 2021
@srawlins
srawlins deleted the bump-test-api branch March 11, 2021 17:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants