Skip to content

Commit a683584

Browse files
[ci] Continue .cirrus.yml alignment with flutter/plugins (#2832)
* Enable more repo checks * Remove Android from Pigeon iOS test app * Add various mostly-no-op tasks * Remove unused Java generation in ObjC test
1 parent 4419bdf commit a683584

File tree

25 files changed

+215
-796
lines changed

25 files changed

+215
-796
lines changed

.cirrus.yml

+48-13
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,7 @@ task:
107107
# has been fixed, this can be removed and there can just be a single
108108
# run with --require-excerpts and no exclusions.
109109
- ./script/tool_runner.sh readme-check --require-excerpts --exclude=script/configs/temp_exclude_excerpt.yaml
110+
dependabot_script: $PLUGIN_TOOL_COMMAND dependabot-check
110111
version_script:
111112
# For pre-submit, pass the PR labels to the script to allow for
112113
# check overrides.
@@ -119,6 +120,14 @@ task:
119120
- ./script/tool_runner.sh version-check --check-for-missing-changes --pr-labels="$CIRRUS_PR_LABELS"
120121
- fi
121122
publishable_script: ./script/tool_runner.sh publish-check --allow-pre-release
123+
federated_safety_script:
124+
# This check is only meaningful for PRs, as it validates changes
125+
# rather than state.
126+
- if [[ $CIRRUS_PR == "" ]]; then
127+
- ./script/tool_runner.sh federation-safety-check
128+
- else
129+
- echo "Only run in presubmit"
130+
- fi
122131
- name: dart_unit_tests
123132
env:
124133
matrix:
@@ -220,6 +229,19 @@ task:
220229
CHANNEL: "master"
221230
CHANNEL: "stable"
222231
<< : *BUILD_ALL_PACKAGES_APP_TEMPLATE
232+
- name: linux-platform_tests
233+
# Don't run full platform tests on both channels in pre-submit.
234+
skip: $CIRRUS_PR != '' && $CHANNEL == 'stable'
235+
env:
236+
matrix:
237+
CHANNEL: "master"
238+
CHANNEL: "stable"
239+
build_script:
240+
- ./script/tool_runner.sh build-examples --linux
241+
native_test_script:
242+
- xvfb-run ./script/tool_runner.sh native-test --linux --no-integration
243+
drive_script:
244+
- xvfb-run ./script/tool_runner.sh drive-examples --linux
223245

224246
# Heavy-workload Linux tasks.
225247
# These use machines with more CPUs and memory, so will reduce parallelization
@@ -251,8 +273,16 @@ task:
251273
# extension_google_sign_in_as_googleapis_auth is currently not building, see
252274
# https://github.com/flutter/flutter/issues/89301
253275
- ./script/tool_runner.sh build-examples --apk --exclude=extension_google_sign_in_as_googleapis_auth
276+
lint_script:
277+
- ./script/tool_runner.sh lint-android # must come after build-examples
254278
native_unit_test_scipt:
255279
- ./script/tool_runner.sh native-test --android --no-integration
280+
# Upload the full lint results to Cirrus to display in the results UI.
281+
always:
282+
android-lint_artifacts:
283+
path: "**/reports/lint-results-debug.xml"
284+
type: text/xml
285+
format: android-lint
256286
- name: android-build_all_packages
257287
env:
258288
BUILD_ALL_ARGS: "apk"
@@ -290,25 +320,13 @@ task:
290320
- cd ../..
291321
- flutter packages get
292322
- dart testing/web_benchmarks_test.dart
293-
### Linux desktop tasks ###
294-
- name: linux-platform_tests
295-
# Don't run full platform tests on both channels in pre-submit.
296-
skip: $CIRRUS_PR != '' && $CHANNEL == 'stable'
297-
env:
298-
matrix:
299-
CHANNEL: "master"
300-
CHANNEL: "stable"
301-
build_script:
302-
- ./script/tool_runner.sh build-examples --linux
303-
native_test_script:
304-
- ./script/tool_runner.sh native-test --linux --no-integration
305323

324+
# ARM macOS tasks.
306325
task:
307326
<< : *FLUTTER_UPGRADE_TEMPLATE
308327
<< : *MACOS_ARM_TEMPLATE
309328
matrix:
310329
- name: macos-custom_package_tests
311-
# Run on macOS x64 image with Java runtime installed.
312330
env:
313331
PATH: $PATH:/usr/local/bin
314332
matrix:
@@ -340,6 +358,16 @@ task:
340358
- ./script/tool_runner.sh build-examples --ios
341359
native_test_script:
342360
- ./script/tool_runner.sh native-test --ios --ios-destination "platform=iOS Simulator,name=iPhone 13,OS=latest"
361+
drive_script:
362+
# `drive-examples` contains integration tests, which changes the UI of the application.
363+
# This UI change sometimes affects `xctest`.
364+
# So we run `drive-examples` after `native-test`; changing the order will result ci failure.
365+
- ./script/tool_runner.sh drive-examples --ios
366+
xcode_analyze_script:
367+
- ./script/tool_runner.sh xcode-analyze --ios
368+
xcode_analyze_deprecation_script:
369+
# Ensure we don't accidentally introduce deprecated code.
370+
- ./script/tool_runner.sh xcode-analyze --ios --ios-min-version=13.0
343371
### iOS tasks ###
344372
- name: ios-build_all_packages
345373
env:
@@ -359,8 +387,15 @@ task:
359387
PATH: $PATH:/usr/local/bin
360388
build_script:
361389
- ./script/tool_runner.sh build-examples --macos
390+
xcode_analyze_script:
391+
- ./script/tool_runner.sh xcode-analyze --macos
392+
xcode_analyze_deprecation_script:
393+
# Ensure we don't accidentally introduce deprecated code.
394+
- ./script/tool_runner.sh xcode-analyze --macos --macos-min-version=12.3
362395
native_test_script:
363396
- ./script/tool_runner.sh native-test --macos
397+
drive_script:
398+
- ./script/tool_runner.sh drive-examples --macos
364399

365400
# Intel macOS tasks.
366401
task:

.github/dependabot.yml

+167
Original file line numberDiff line numberDiff line change
@@ -22,3 +22,170 @@ updates:
2222
labels:
2323
- "team"
2424
- "team: infra"
25+
26+
- package-ecosystem: "gradle"
27+
directory: "/packages/animations/example/android/app"
28+
commit-message:
29+
prefix: "[animations]"
30+
schedule:
31+
interval: "weekly"
32+
open-pull-requests-limit: 10
33+
ignore:
34+
- dependency-name: "*"
35+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
36+
- package-ecosystem: "gradle"
37+
directory: "/packages/dynamic_layouts/example/android/app"
38+
commit-message:
39+
prefix: "[dynamic_lyts]"
40+
schedule:
41+
interval: "weekly"
42+
open-pull-requests-limit: 10
43+
ignore:
44+
- dependency-name: "*"
45+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
46+
- package-ecosystem: "gradle"
47+
directory: "/packages/extension_google_sign_in_as_googleapis_auth/example/android/app"
48+
commit-message:
49+
prefix: "[ext_sign_in]"
50+
schedule:
51+
interval: "weekly"
52+
open-pull-requests-limit: 10
53+
ignore:
54+
- dependency-name: "*"
55+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
56+
- package-ecosystem: "gradle"
57+
directory: "/packages/flutter_adaptive_scaffold/example/android/app"
58+
commit-message:
59+
prefix: "[scaffold]"
60+
schedule:
61+
interval: "weekly"
62+
open-pull-requests-limit: 10
63+
ignore:
64+
- dependency-name: "*"
65+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
66+
- package-ecosystem: "gradle"
67+
directory: "/packages/flutter_markdown/example/android/app"
68+
commit-message:
69+
prefix: "[markdown]"
70+
schedule:
71+
interval: "weekly"
72+
open-pull-requests-limit: 10
73+
ignore:
74+
- dependency-name: "*"
75+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
76+
- package-ecosystem: "gradle"
77+
directory: "/packages/go_router/example/android/app"
78+
commit-message:
79+
prefix: "[go_router]"
80+
schedule:
81+
interval: "weekly"
82+
open-pull-requests-limit: 10
83+
ignore:
84+
- dependency-name: "*"
85+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
86+
- package-ecosystem: "gradle"
87+
directory: "/packages/imitation_game/imitation_tests/smiley/flutter/smiley/android/app"
88+
commit-message:
89+
prefix: "[imitation]"
90+
schedule:
91+
interval: "weekly"
92+
open-pull-requests-limit: 10
93+
ignore:
94+
- dependency-name: "*"
95+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
96+
- package-ecosystem: "gradle"
97+
directory: "/packages/palette_generator/example/android/app"
98+
commit-message:
99+
prefix: "[palette_gen]"
100+
schedule:
101+
interval: "weekly"
102+
open-pull-requests-limit: 10
103+
ignore:
104+
- dependency-name: "*"
105+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
106+
- package-ecosystem: "gradle"
107+
directory: "/packages/pigeon/platform_tests/test_plugin/android"
108+
commit-message:
109+
prefix: "[pigeon]"
110+
schedule:
111+
interval: "weekly"
112+
open-pull-requests-limit: 10
113+
ignore:
114+
- dependency-name: "com.android.tools.build:gradle"
115+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
116+
- dependency-name: "junit:junit"
117+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
118+
- dependency-name: "org.mockito:*"
119+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
120+
- dependency-name: "androidx.test:*"
121+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
122+
- dependency-name: "org.robolectric:*"
123+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
124+
- package-ecosystem: "gradle"
125+
directory: "/packages/pigeon/platform_tests/test_plugin/example/android/app"
126+
commit-message:
127+
prefix: "[pigeon]"
128+
schedule:
129+
interval: "weekly"
130+
open-pull-requests-limit: 10
131+
ignore:
132+
- dependency-name: "*"
133+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
134+
- package-ecosystem: "gradle"
135+
directory: "/packages/pigeon/platform_tests/alternate_language_test_plugin/android"
136+
commit-message:
137+
prefix: "[pigeon]"
138+
schedule:
139+
interval: "weekly"
140+
open-pull-requests-limit: 10
141+
ignore:
142+
- dependency-name: "com.android.tools.build:gradle"
143+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
144+
- dependency-name: "junit:junit"
145+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
146+
- dependency-name: "org.mockito:*"
147+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
148+
- dependency-name: "androidx.test:*"
149+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
150+
- dependency-name: "org.robolectric:*"
151+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
152+
- package-ecosystem: "gradle"
153+
directory: "/packages/pigeon/platform_tests/alternate_language_test_plugin/example/android/app"
154+
commit-message:
155+
prefix: "[pigeon]"
156+
schedule:
157+
interval: "weekly"
158+
open-pull-requests-limit: 10
159+
ignore:
160+
- dependency-name: "*"
161+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
162+
- package-ecosystem: "gradle"
163+
directory: "/packages/rfw/example/hello/android/app"
164+
commit-message:
165+
prefix: "[rfw]"
166+
schedule:
167+
interval: "weekly"
168+
open-pull-requests-limit: 10
169+
ignore:
170+
- dependency-name: "*"
171+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
172+
- package-ecosystem: "gradle"
173+
directory: "/packages/rfw/example/local/android/app"
174+
commit-message:
175+
prefix: "[rfw]"
176+
schedule:
177+
interval: "weekly"
178+
open-pull-requests-limit: 10
179+
ignore:
180+
- dependency-name: "*"
181+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]
182+
- package-ecosystem: "gradle"
183+
directory: "/packages/rfw/example/remote/android/app"
184+
commit-message:
185+
prefix: "[rfw]"
186+
schedule:
187+
interval: "weekly"
188+
open-pull-requests-limit: 10
189+
ignore:
190+
- dependency-name: "*"
191+
update-types: ["version-update:semver-minor", "version-update:semver-patch"]

packages/pigeon/e2e_tests/test_objc/android/.project

-28
This file was deleted.

packages/pigeon/e2e_tests/test_objc/android/.settings/org.eclipse.buildship.core.prefs

-13
This file was deleted.

packages/pigeon/e2e_tests/test_objc/android/app/.classpath

-6
This file was deleted.

packages/pigeon/e2e_tests/test_objc/android/app/.project

-34
This file was deleted.

packages/pigeon/e2e_tests/test_objc/android/app/.settings/org.eclipse.buildship.core.prefs

-2
This file was deleted.

0 commit comments

Comments
 (0)