Skip to content

Commit 0f2f62b

Browse files
[ci] Add partial LUCI version of repo_checks (flutter#4389)
Adds a bringup-mode version of the parts of `repo_checks` that can currently be run on LUCI. When this is enabled in a follow-up I'll remove the relevant checks from the Cirrus version, leaving `repo_checks` split across the two systems for now, but without duplication. This has a slight overhead in terms of machine setup, but allow better isolating the parts of the CI that are still actually blocked from migration. Part of flutter#114373
1 parent b267e31 commit 0f2f62b

File tree

2 files changed

+66
-0
lines changed

2 files changed

+66
-0
lines changed

.ci.yaml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@ platform_properties:
1414
os: Ubuntu
1515
cores: "8"
1616
device_type: none
17+
dependencies: >-
18+
[
19+
{"dependency": "curl", "version": "version:7.64.0"}
20+
]
1721
linux_android:
1822
properties:
1923
os: Ubuntu
@@ -92,6 +96,21 @@ targets:
9296
channel: master
9397
version_file: flutter_master.version
9498

99+
- name: Linux repo_checks
100+
bringup: true # New target
101+
recipe: packages/packages
102+
timeout: 30
103+
properties:
104+
add_recipes_cq: "true"
105+
target_file: repo_checks.yaml
106+
channel: master
107+
version_file: flutter_master.version
108+
# The format check requires clang-format.
109+
dependencies: >-
110+
[
111+
{"dependency": "clang", "version": "git_revision:5d5aba78dbbee75508f01bcaa69aedb2ab79065a"}
112+
]
113+
95114
- name: Linux dart_unit_test_shard_1 master
96115
recipe: packages/packages
97116
timeout: 60

.ci/targets/repo_checks.yaml

Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
tasks:
2+
- name: prepare tool
3+
script: .ci/scripts/prepare_tool.sh
4+
- name: format
5+
script: script/tool_runner.sh
6+
args: ["format", "--fail-on-change"]
7+
always: true
8+
- name: license validation
9+
script: script/tool_runner.sh
10+
args: ["license-check"]
11+
always: true
12+
# The major and minor version here should match the lowest version analyzed
13+
# in legacy version analysis (currently .cirrus.yaml legacy_version_analyze).
14+
- name: pubspec validation
15+
script: script/tool_runner.sh
16+
args:
17+
- "pubspec-check"
18+
- "--min-min-flutter-version=3.3.0"
19+
- "--allow-dependencies=script/configs/allowed_unpinned_deps.yaml"
20+
- "--allow-pinned-dependencies=script/configs/allowed_pinned_deps.yaml"
21+
always: true
22+
- name: README validation
23+
script: script/tool_runner.sh
24+
args: ["readme-check"]
25+
always: true
26+
# Re-run with --require-excerpts, skipping packages that still need
27+
# to be converted. Once https://github.com/flutter/flutter/issues/102679
28+
# has been fixed, this can be removed --require-excerpts added to the
29+
# run above.
30+
- name: README snippet validation
31+
script: script/tool_runner.sh
32+
args: ["readme-check", "--require-excerpts", "--exclude=script/configs/temp_exclude_excerpt.yaml"]
33+
always: true
34+
- name: Gradle validation
35+
script: script/tool_runner.sh
36+
args: ["gradle-check"]
37+
always: true
38+
- name: Dependabot coverage validation
39+
script: script/tool_runner.sh
40+
args: ["dependabot-check"]
41+
always: true
42+
- name: publishability
43+
script: script/tool_runner.sh
44+
args: ["publish-check", "--allow-pre-release"]
45+
always: true
46+
# TODO(stuartmorgan): Convert remaining checks from Cirrus repo_checks. See
47+
# https://github.com/flutter/flutter/issues/114373

0 commit comments

Comments
 (0)