Skip to content

Turn on bitcode for integration tests and add-to-app templates#44633

Merged
jmagman merged 1 commit intoflutter:masterfrom
jmagman:bitcode-on
Nov 13, 2019
Merged

Turn on bitcode for integration tests and add-to-app templates#44633
jmagman merged 1 commit intoflutter:masterfrom
jmagman:bitcode-on

Conversation

@jmagman
Copy link
Member

@jmagman jmagman commented Nov 12, 2019

This reverts commit 41c02d7.

Description

Turn back on bitcode for integration tests and the add-to-app templates.

Related Issues

Installation bitcode issues seem to be fixed with flutter/engine#13428.

@jmagman jmagman added a: tests "flutter test", flutter_test, or one of our tests tool Affects the "flutter" command-line tool. See also t: labels. a: existing-apps Integration with existing apps via the add-to-app flow labels Nov 12, 2019
@jmagman jmagman requested a review from dnfield November 12, 2019 02:47
@fluttergithubbot fluttergithubbot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Nov 12, 2019
Copy link
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

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

This LGTM, although it would be good for @digiter and @chinmaygarde to verify that the devicelab and engine builders will stay in sync. As best I can tell, devicelab is currently all on 11.x and one instance of 10.3, and engine builders are on 10.2

@dnfield
Copy link
Contributor

dnfield commented Nov 12, 2019

You may need to merge up to head or something for the arb conflicts? Hans would probably know more.

@jmagman jmagman merged commit 2d42b43 into flutter:master Nov 13, 2019
@jmagman jmagman deleted the bitcode-on branch November 13, 2019 02:00
@jmagman jmagman added the platform-ios iOS applications specifically label Aug 21, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

a: existing-apps Integration with existing apps via the add-to-app flow a: tests "flutter test", flutter_test, or one of our tests c: contributor-productivity Team-specific productivity, code health, technical debt. platform-ios iOS applications specifically tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants