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

Conversation

@matanlurey
Copy link
Contributor

Closes flutter/flutter#93261.

I probably could have been more clever with a regular-expression but this seems easier to read.

@matanlurey matanlurey requested review from dnfield and zanderso August 26, 2023 00:08
@matanlurey
Copy link
Contributor Author

Hmm, this is failing @godofredoc @zanderso:

Protobuf JSON decoding failed at: root["targets"]["5"]["runIfNot"]. Unknown field name 'runIfNot'
runIfNot

@godofredoc
Copy link
Contributor

Edit
Hide
Delete

Hmm, this is failing @godofredoc @zanderso:

Protobuf JSON decoding failed at: root["targets"]["5"]["runIfNot"]. Unknown field name 'runIfNot'
runIfNot

Checking, it seems like I missed updating the proto somewhere.

1 similar comment
@godofredoc
Copy link
Contributor

Edit
Hide
Delete

Hmm, this is failing @godofredoc @zanderso:

Protobuf JSON decoding failed at: root["targets"]["5"]["runIfNot"]. Unknown field name 'runIfNot'
runIfNot

Checking, it seems like I missed updating the proto somewhere.

@godofredoc
Copy link
Contributor

Checking, it seems like I missed updating the proto somewhere.
Auto deployment of the backend is disabled. Seems like it was disabled because changes in the dart - data store module broke transaction management.

@godofredoc
Copy link
Contributor

Found the issue for the backend deployment flutter/cocoon#3009

@godofredoc
Copy link
Contributor

Blocked on flutter/flutter#133376

@godofredoc
Copy link
Contributor

@matanlurey this is unblocked now.

@godofredoc godofredoc removed their assignment Aug 29, 2023
@matanlurey
Copy link
Contributor Author

Thank you!!!

@matanlurey
Copy link
Contributor Author

Ping @zanderso @dnfield this looks ready for review. Thanks!

@matanlurey matanlurey merged commit 5fac67f into flutter:main Aug 29, 2023
@matanlurey matanlurey deleted the engine-fuchsia-run-if-not branch August 29, 2023 18:34
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 29, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Aug 29, 2023
gaaclarke pushed a commit to gaaclarke/engine that referenced this pull request Aug 30, 2023
…s. (flutter#45134)

Closes flutter/flutter#93261.

I probably could have been more clever with a regular-expression but
this seems easier to read.

---------

Co-authored-by: godofredoc <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[engine] Linux Fuchsia and Linux Fuchsia FEMU don't have runif logic

3 participants