Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.
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
2 changes: 0 additions & 2 deletions .clang-tidy
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
# A YAML format of https://clang.llvm.org/extra/clang-tidy/.

# Prefix check with "-" to ignore.
# Note: Some of the checks here are used as errors selectively, see
# //ci/lint.sh
Checks: >-
bugprone-use-after-move,
bugprone-unchecked-optional-access,
Expand Down
10 changes: 5 additions & 5 deletions ci/builders/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -464,7 +464,7 @@ Engine test example:
{
"tests": [
{
"name": "test: lint android_debug_arm64",
"name": "test: clang_tidy android_debug_arm64",
"recipe": "engine_v2/tester_engine",
"drone_dimensions": [
"device_type=none",
Expand All @@ -476,7 +476,7 @@ Engine test example:
],
"tasks": [
{
"name": "test: lint android_debug_arm64",
"name": "test: clang_tidy android_debug_arm64",
"parameters": [
"--variant",
"android_debug_arm64",
Expand All @@ -485,7 +485,7 @@ Engine test example:
"--shard-variants=host_debug"
],
"max_attempts": 1,
"script": "flutter/ci/lint.sh"
"script": "flutter/ci/clang_tidy.sh"
}
]
}
Expand Down Expand Up @@ -513,7 +513,7 @@ Example task configuration:

```json
{
"name": "test: lint android_debug_arm64",
"name": "test: clang_tidy android_debug_arm64",
"parameters": [
"--variant",
"android_debug_arm64",
Expand All @@ -522,7 +522,7 @@ Example task configuration:
"--shard-variants=host_debug"
],
"max_attempts": 1,
"script": "flutter/ci/lint.sh"
"script": "flutter/ci/clang_tidy.sh"
}
```

Expand Down
6 changes: 5 additions & 1 deletion ci/clang_tidy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,14 @@ DART="${DART_BIN}/dart"
# FLUTTER_LINT_PRINT_FIX will make it so that fix is executed and the generated
# diff is printed to stdout if clang-tidy fails. This is helpful for enabling
# new lints.

# To run on CI, just uncomment the following line:
# FLUTTER_LINT_PRINT_FIX=1

if [[ -z "${FLUTTER_LINT_PRINT_FIX}" ]]; then
fix_flag=""
else
fix_flag="--fix"
fix_flag="--fix --lint-all"
fi

COMPILE_COMMANDS="$SRC_DIR/out/host_debug/compile_commands.json"
Expand Down
35 changes: 0 additions & 35 deletions ci/lint.sh

This file was deleted.

4 changes: 2 additions & 2 deletions tools/githooks/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@ scripts have the names that `git` will look for.
This hooks runs when pushing commits to a remote branch, for example to
create or update a pull request: `git push origin my-local-branch`.

The `pre-push` hook runs `ci/lint.sh` and `ci/format.sh`. `ci/analyze.sh` and
`ci/licenses.sh` are more expensive and are not run.
The `pre-push` hook runs `ci/clang_tidy.sh`, `ci/pylint.sh` and `ci/format.sh`.
`ci/analyze.sh` and `ci/licenses.sh` are more expensive and are not run.

### Adding new pre-push checks

Expand Down