Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit b83e0aa

Browse files
zandersozanderso
committed
Adds a linux_clang_tidy_presubmit builder
1 parent 48558a3 commit b83e0aa

File tree

3 files changed

+249
-0
lines changed

3 files changed

+249
-0
lines changed

‎.ci.yaml‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -226,6 +226,25 @@ targets:
226226
- "**.frag"
227227
- "**.vert"
228228

229+
- name: Linux linux_clang_tidy_presubmit
230+
bringup: true
231+
recipe: engine_v2/engine_v2
232+
timeout: 120
233+
properties:
234+
config_name: linux_clang_tidy_presubmit
235+
runIf:
236+
- DEPS
237+
- .ci.yaml
238+
- tools/clang_tidy/**
239+
- ci/builders/**
240+
- ci/clang_tidy.sh
241+
- "**.h"
242+
- "**.c"
243+
- "**.cc"
244+
- "**.fbs"
245+
- "**.frag"
246+
- "**.vert"
247+
229248
- name: Linux linux_arm_host_engine
230249
recipe: engine_v2/engine_v2
231250
timeout: 120
@@ -360,6 +379,27 @@ targets:
360379
- "**.m"
361380
- "**.mm"
362381

382+
- name: Mac mac_clang_tidy_presubmit
383+
bringup: true
384+
recipe: engine_v2/engine_v2
385+
timeout: 120
386+
properties:
387+
config_name: mac_clang_tidy_presubmit
388+
runIf:
389+
- DEPS
390+
- .ci.yaml
391+
- tools/clang_tidy/**
392+
- ci/builders/**
393+
- ci/clang_tidy.sh
394+
- "**.h"
395+
- "**.c"
396+
- "**.cc"
397+
- "**.fbs"
398+
- "**.frag"
399+
- "**.vert"
400+
- "**.m"
401+
- "**.mm"
402+
363403
- name: Mac mac_host_engine
364404
recipe: engine_v2/engine_v2
365405
timeout: 120

‎ci/builders/linux_clang_tidy_presubmit.json‎

Lines changed: 95 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,95 @@
1+
{
2+
"builds": [
3+
{
4+
"name": "android_debug_arm64",
5+
"archives": [],
6+
"drone_dimensions": [
7+
"device_type=none",
8+
"os=Linux"
9+
],
10+
"gn": [
11+
"--android",
12+
"--android-cpu",
13+
"arm64",
14+
"--no-lto"
15+
],
16+
"ninja": {
17+
"config": "android_debug_arm64"
18+
}
19+
},
20+
{
21+
"name": "host_debug",
22+
"archives": [],
23+
"drone_dimensions": [
24+
"device_type=none",
25+
"os=Linux"
26+
],
27+
"gn": [
28+
"--runtime-mode",
29+
"debug",
30+
"--prebuilt-dart-sdk",
31+
"--no-lto"
32+
],
33+
"ninja": {
34+
"config": "host_debug"
35+
}
36+
}
37+
],
38+
"tests": [
39+
{
40+
"name": "test: lint host_debug",
41+
"recipe": "engine_v2/tester_engine",
42+
"drone_dimensions": [
43+
"device_type=none",
44+
"os=Linux",
45+
"cores=32"
46+
],
47+
"gclient_variables": {
48+
"download_android_deps": false
49+
},
50+
"dependencies": [
51+
"host_debug",
52+
"android_debug_arm64"
53+
],
54+
"tasks": [
55+
{
56+
"name": "test: lint host_debug",
57+
"parameters": [
58+
"--variant",
59+
"host_debug",
60+
"--shard-id=0",
61+
"--shard-variants=android_debug_arm64"
62+
],
63+
"max_attempts": 1,
64+
"script": "flutter/ci/clang_tidy.sh"
65+
}
66+
]
67+
},
68+
{
69+
"name": "test: lint android_debug_arm64",
70+
"recipe": "engine_v2/tester_engine",
71+
"drone_dimensions": [
72+
"device_type=none",
73+
"os=Linux",
74+
"cores=32"
75+
],
76+
"dependencies": [
77+
"host_debug",
78+
"android_debug_arm64"
79+
],
80+
"tasks": [
81+
{
82+
"name": "test: lint android_debug_arm64",
83+
"parameters": [
84+
"--variant",
85+
"android_debug_arm64",
86+
"--shard-id=1",
87+
"--shard-variants=host_debug"
88+
],
89+
"max_attempts": 1,
90+
"script": "flutter/ci/clang_tidy.sh"
91+
}
92+
]
93+
}
94+
]
95+
}

‎ci/builders/mac_clang_tidy_presubmit.json‎

Lines changed: 114 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,114 @@
1+
{
2+
"builds": [
3+
{
4+
"drone_dimensions": [
5+
"device_type=none",
6+
"os=Mac-12",
7+
"cpu=arm64"
8+
],
9+
"gclient_variables": {
10+
"download_android_deps": false
11+
},
12+
"gn": [
13+
"--runtime-mode",
14+
"debug",
15+
"--prebuilt-dart-sdk",
16+
"--no-lto",
17+
"--force-mac-arm64"
18+
],
19+
"name": "host_debug",
20+
"ninja": {
21+
"config": "host_debug"
22+
}
23+
},
24+
{
25+
"drone_dimensions": [
26+
"device_type=none",
27+
"os=Mac-12",
28+
"cpu=arm64"
29+
],
30+
"gclient_variables": {
31+
"download_android_deps": false
32+
},
33+
"gn": [
34+
"--ios",
35+
"--runtime-mode",
36+
"debug",
37+
"--simulator",
38+
"--no-lto",
39+
"--force-mac-arm64"
40+
],
41+
"name": "ios_debug_sim",
42+
"ninja": {
43+
"config": "ios_debug_sim"
44+
}
45+
}
46+
],
47+
"tests": [
48+
{
49+
"name": "test: lint host_debug",
50+
"recipe": "engine_v2/tester_engine",
51+
"drone_dimensions": [
52+
"device_type=none",
53+
"os=Mac",
54+
"cpu=arm64"
55+
],
56+
"gclient_variables": {
57+
"download_android_deps": false
58+
},
59+
"dependencies": [
60+
"host_debug",
61+
"ios_debug_sim"
62+
],
63+
"contexts": [
64+
"osx_sdk"
65+
],
66+
"tasks": [
67+
{
68+
"name": "test: lint host_debug",
69+
"parameters": [
70+
"--variant",
71+
"host_debug",
72+
"--shard-id=0",
73+
"--shard-variants=ios_debug_sim"
74+
],
75+
"max_attempts": 1,
76+
"script": "flutter/ci/clang_tidy.sh"
77+
}
78+
]
79+
},
80+
{
81+
"name": "test: lint ios_debug_sim",
82+
"recipe": "engine_v2/tester_engine",
83+
"drone_dimensions": [
84+
"device_type=none",
85+
"os=Mac",
86+
"cpu=arm64"
87+
],
88+
"gclient_variables": {
89+
"download_android_deps": false
90+
},
91+
"dependencies": [
92+
"host_debug",
93+
"ios_debug_sim"
94+
],
95+
"contexts": [
96+
"osx_sdk"
97+
],
98+
"tasks": [
99+
{
100+
"name": "test: lint ios_debug_sim",
101+
"parameters": [
102+
"--variant",
103+
"ios_debug_sim",
104+
"--lint-all",
105+
"--shard-id=1",
106+
"--shard-variants=host_debug"
107+
],
108+
"max_attempts": 1,
109+
"script": "flutter/ci/clang_tidy.sh"
110+
}
111+
]
112+
}
113+
]
114+
}

0 commit comments

Comments
 (0)