Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Enable analysis for the tool directory #3853

Merged
merged 3 commits into from
May 7, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,6 @@ task:
- name: format
format_script: ./script/incremental_build.sh format --fail-on-change
license_script:
- cd script/tool
- pub get
- cd ../..
- dart script/tool/lib/src/main.dart license-check
- name: test
env:
Expand All @@ -72,7 +69,10 @@ task:
env:
matrix:
CHANNEL: "master"
script:
tool_script:
- cd script/tool
- dart analyze --fatal-infos
plugins_script:
- ./script/incremental_build.sh analyze
## TODO(cyanglaz):
## Combing stable and master analyze jobs when integration test null safety is ready on flutter stable.
Expand Down
6 changes: 3 additions & 3 deletions script/tool/lib/src/drive_examples_command.dart
Original file line number Diff line number Diff line change
Expand Up @@ -101,13 +101,13 @@ class DriveExamplesCommand extends PluginCommand {
fileSystem.directory(p.join(example.path, 'integration_test'));

if (await integrationTests.exists()) {
await for (final FileSystemEntity integration_test
await for (final FileSystemEntity integrationTest
in integrationTests.list()) {
if (!integration_test.basename.endsWith('_test.dart')) {
if (!integrationTest.basename.endsWith('_test.dart')) {
continue;
}
targetPaths
.add(p.relative(integration_test.path, from: example.path));
.add(p.relative(integrationTest.path, from: example.path));
}
}

Expand Down
2 changes: 1 addition & 1 deletion script/tool/lib/src/version_check_command.dart
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ class VersionCheckCommand extends PluginCommand {

// Skip validation for the special NEXT version that's used to accumulate
// changes that don't warrant publishing on their own.
bool hasNextSection = versionString == 'NEXT';
final bool hasNextSection = versionString == 'NEXT';
if (hasNextSection) {
print('Found NEXT; validating next version in the CHANGELOG.');
// Ensure that the version in pubspec hasn't changed without updating
Expand Down