This repository was archived by the owner on Feb 22, 2023. It is now read-only.
Update CI config for Flutter 2 #3674
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Remove the temporarily-added beta branch testing. - Enable Linux and macOS on stable. - Combine the two different macOS matrix configurations now that they are the same. - Eliminate the NNBD plugin filtering for stable.
|
Looks like there are issues with regenerating the Docker instances that would need investigation and reworking, so reverting the merging of the Dockerfiles. |
This reverts commit 624f134.
blasten
reviewed
Mar 4, 2021
Comment on lines
+126
to
+128
| # These use machines with more CPUs and memory, so will reduce parallelization | ||
| # for non-credit runs. | ||
| task: |
There was a problem hiding this comment.
does it need cpu and memory values under container?
There was a problem hiding this comment.
No, there are defaults (2 CPUs and 4 GB)
There was a problem hiding this comment.
I was wondering what makes these tasks use machines with more CPUs and memory as mentioned in the comment above?
There was a problem hiding this comment.
🤦
I didn't look carefully enough at the context and thought this was about the light-load config. Fix incoming.
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
Mar 4, 2021
fluttergithubbot
pushed a commit
to flutter/flutter
that referenced
this pull request
Mar 4, 2021
NickalasB
added a commit
to NickalasB/plugins
that referenced
this pull request
Mar 8, 2021
* master: [google_sign_in] fix test(flutter#3690) [extension_google_sign_in_as_googleapis_auth] Update import (flutter#3689) [google_sign_in] Updates google_sign_in_platform_interfaces adding parametrized `clientId` (flutter#3686) Import flutter_test for future compatibility (flutter#3665) [ci] Disable analyze on stable for web plugins that contains null safety integration tests. (flutter#3681) Bring HTML inputs into view automatically (flutter#3655) [in_app_purchase] presentCodeRedemptionSheet (flutter#3274) [google_maps_flutter_web] Downgrade mockito in example app. (flutter#3679) Update CI config for Flutter 2 (flutter#3674) [image_picker] fix flutter/flutter#71927 (flutter#3676) [google_maps_flutter_web] Move integration tests to example. (flutter#3675) [google_maps_flutter_web] Make google_maps_flutter_web work with latest plugins (flutter#3673) # Conflicts: # packages/webview_flutter/CHANGELOG.md
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Includes cleanup to simplify our setup. Major changes:
are the same.
Dockerfile (which now also includes clang-format)
now renamed, because the driver installer script doesn't support
Chrome 89 yet.
tasks to run in parallel without hitting the community limit.
platform groupings
tests