Skip to content

Commit 724f52d

Browse files
[ci] Reduce tasks slightly (#6011)
Help buy a bit of time until we eliminate the max number of tasks for `release` to wait for, and be more efficient with CI resources: - Folds the very fast repo-tooling-unit-test step into the existing repo checks task, cutting down on machine overhead. - Removes the `master` version of downgraded analysis, since it's hard to imagine a plausible scenario where it fails but the `stable` version does not, so we don't need to spend resources on both. See flutter/flutter#141196
1 parent 73c9bdc commit 724f52d

File tree

2 files changed

+6
-27
lines changed

2 files changed

+6
-27
lines changed

.ci.yaml

Lines changed: 4 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -107,20 +107,6 @@ platform_properties:
107107
}
108108
109109
targets:
110-
### Linux-host general tasks ###
111-
- name: Linux repo_tools_tests
112-
recipe: packages/packages
113-
timeout: 30
114-
properties:
115-
add_recipes_cq: "true"
116-
target_file: repo_tools_tests.yaml
117-
channel: master
118-
version_file: flutter_master.version
119-
env_variables: >-
120-
{
121-
"CHANNEL": "master"
122-
}
123-
124110
- name: Linux repo_checks
125111
recipe: packages/packages
126112
timeout: 30
@@ -281,19 +267,10 @@ targets:
281267
"CHANNEL": "stable"
282268
}
283269
284-
- name: Linux analyze_downgraded master
285-
recipe: packages/packages
286-
timeout: 30
287-
properties:
288-
add_recipes_cq: "true"
289-
target_file: analyze_downgraded.yaml
290-
channel: master
291-
version_file: flutter_master.version
292-
env_variables: >-
293-
{
294-
"CHANNEL": "master"
295-
}
296-
270+
# This is only run on stable since it's extremely likely that stable will
271+
# resolve to packages that are older (or at least as old) than on master, so
272+
# running a second copy with master is very unlikely to catch anything that
273+
# this doesn't.
297274
- name: Linux analyze_downgraded stable
298275
recipe: packages/packages
299276
timeout: 30

.ci/targets/repo_checks.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@ tasks:
22
- name: prepare tool
33
script: .ci/scripts/prepare_tool.sh
44
infra_step: true # Note infra steps failing prevents "always" from running.
5+
- name: tool unit tests
6+
script: .ci/scripts/plugin_tools_tests.sh
57
- name: format
68
script: script/tool_runner.sh
79
# Skip Swift formatting on Linux builders.

0 commit comments

Comments
 (0)