Skip to content

Commit 95bc1c6

Browse files
[ci] Switch analyze tests to LUCI (flutter#4264)
Fully enables the new analyze and downgraded-analyze tests in LUCI, and removes the Cirrus versions. Part of flutter/flutter#114373
1 parent 24ad5a4 commit 95bc1c6

File tree

2 files changed

+2
-37
lines changed

2 files changed

+2
-37
lines changed

.ci.yaml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,16 +91,15 @@ targets:
9191
version_file: flutter_master.version
9292

9393
- name: Linux analyze master
94-
bringup: true # New target
9594
recipe: packages/packages
9695
timeout: 30
9796
properties:
97+
add_recipes_cq: "true"
9898
target_file: analyze.yaml
9999
channel: master
100100
version_file: flutter_master.version
101101

102102
- name: Linux analyze stable
103-
bringup: true # New target
104103
recipe: packages/packages
105104
timeout: 30
106105
properties:
@@ -109,16 +108,15 @@ targets:
109108
version_file: flutter_stable.version
110109

111110
- name: Linux analyze_downgraded master
112-
bringup: true # New target
113111
recipe: packages/packages
114112
timeout: 30
115113
properties:
114+
add_recipes_cq: "true"
116115
target_file: analyze_downgraded.yaml
117116
channel: master
118117
version_file: flutter_master.version
119118

120119
- name: Linux analyze_downgraded stable
121-
bringup: true # New target
122120
recipe: packages/packages
123121
timeout: 30
124122
properties:

.cirrus.yml

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -115,30 +115,6 @@ task:
115115
- else
116116
- ./script/tool_runner.sh federation-safety-check
117117
- fi
118-
- name: analyze
119-
env:
120-
matrix:
121-
CHANNEL: "master"
122-
CHANNEL: "stable"
123-
analyze_tool_script:
124-
- cd script/tool
125-
- dart analyze --fatal-infos
126-
analyze_script:
127-
# DO NOT change the custom-analysis argument here without changing the Dart repo.
128-
# See the comment in script/configs/custom_analysis.yaml for details.
129-
- ./script/tool_runner.sh analyze --custom-analysis=script/configs/custom_analysis.yaml
130-
pathified_analyze_script:
131-
# Re-run analysis with path-based dependencies to ensure that publishing
132-
# the changes won't break analysis of other packages in the respository
133-
# that depend on it.
134-
- ./script/tool_runner.sh make-deps-path-based --target-dependencies-with-non-breaking-updates
135-
# This uses --run-on-dirty-packages rather than --packages-for-branch
136-
# since only the packages changed by 'make-deps-path-based' need to be
137-
# checked.
138-
- $PLUGIN_TOOL_COMMAND analyze --run-on-dirty-packages --log-timing --custom-analysis=script/configs/custom_analysis.yaml
139-
# Restore the tree to a clean state, to avoid accidental issues if
140-
# other script steps are added to this task.
141-
- git checkout .
142118
# Does a sanity check that packages at least pass analysis on the N-1 and N-2
143119
# versions of Flutter stable if the package claims to support that version.
144120
# This is to minimize accidentally making changes that break old versions
@@ -147,7 +123,6 @@ task:
147123
# Note: The versions below should be manually updated after a new stable
148124
# version comes out.
149125
- name: legacy_version_analyze
150-
depends_on: analyze
151126
matrix:
152127
# Change the arguments to pubspec-check when changing these values.
153128
env:
@@ -164,14 +139,6 @@ task:
164139
# Only analyze lib/; non-client code doesn't need to work on
165140
# all supported legacy version.
166141
- ./script/tool_runner.sh analyze --lib-only --skip-if-not-supporting-flutter-version="$CHANNEL" --skip-if-not-supporting-dart-version="$DART_VERSION" --custom-analysis=script/configs/custom_analysis.yaml
167-
# Does a sanity check that packages pass analysis with the lowest possible
168-
# versions of all dependencies. This is to catch cases where we add use of
169-
# new APIs but forget to update minimum versions of dependencies to where
170-
# those APIs are introduced.
171-
- name: downgraded_analyze
172-
depends_on: analyze
173-
analyze_script:
174-
- ./script/tool_runner.sh analyze --downgrade --custom-analysis=script/configs/custom_analysis.yaml
175142
- name: readme_excerpts
176143
env:
177144
CIRRUS_CLONE_SUBMODULES: true

0 commit comments

Comments
 (0)