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

Update OCMock to version 3.7.1 #21111

Merged
merged 1 commit into from
Sep 11, 2020
Merged

Update OCMock to version 3.7.1 #21111

merged 1 commit into from
Sep 11, 2020

Conversation

cbracken
Copy link
Member

@cbracken cbracken commented Sep 11, 2020

This gets us to latest and fixes a warning about a missing cast.

Related patch: flutter/buildroot#401
Related patch: flutter/buildroot#402

@cbracken cbracken changed the title Update OCMocck to version 3.7.1 Update OCMock to version 3.7.1 Sep 11, 2020
@cbracken cbracken requested a review from justinmc September 11, 2020 18:05
Copy link
Contributor

@justinmc justinmc left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@justinmc
Copy link
Contributor

Hmm looks like this is still getting the same failure in the Mac iOS Engine tests. (The other web tests failure is just a timeout).

This gets us to latest and fixes a warning about a missing cast.

This rolls buildroot to f83d1d7

Related patch: flutter/buildroot#401
@justinmc
Copy link
Contributor

justinmc commented Sep 11, 2020

The new buildroot change flutter/buildroot#402 doesn't seem to work with my PR. See my comment flutter/buildroot#402 (comment). Fixed with a gclient sync.

@cbracken cbracken merged commit 5239e55 into flutter:master Sep 11, 2020
@cbracken cbracken deleted the ocmock-371 branch September 11, 2020 20:52
@cbracken
Copy link
Member Author

Update is that flutter/buildroot#403 re-splits these into a source_set and static_library but forcing the static_library to inherit all the sources in the dependency (which is non-standard, but supported for cases like this). That PR has landed in the buildroot -- will get it rolled to the engine once we're done dealing with the P0 flakiness issue in the iOS Scenario tests.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants