summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorMichael Niewöhner <foss@mniewoehner.de>2021-04-09 16:26:32 +0200
committerPatrick Georgi <pgeorgi@google.com>2021-04-13 14:29:43 +0000
commit64d31f48d276aa491976fa7b0a8b1eddf9347ca9 (patch)
tree59af639296ef3a4f0a7f2c93f2278b458c6c8dcb /tests
parent2b4da16ea410c7536ec7cb9f600877ec5ae8e502 (diff)
downloadcoreboot-64d31f48d276aa491976fa7b0a8b1eddf9347ca9.tar.xz
lint: MAINTAINERS: check path matches to not only cover the directory
Gerrit is able to add reviewers based on entries in the `MAINTAINERS` file. For inclusion and exclusion matches either paths or regular expressions can be used. The syntax is described in the header of the file. When matching a path, there are two sensible possibilities: - `path/to/file` matches a file. - `path/to/dir/` matches a folder including its contents recursively. - `path/to/dir/*` matches all files in that folder, without recursing into its subfolders. The trailing slash in the second example is essential. Without it, only the directory entry itself matches when, for example, the folder gets deleted, renamed or its permissions get modified. Reviewers in the list won't get added to changes of any files or directories below that path. Thus, add a linter script to ensure a path match on a directory always ends with `/` or `/*` as shown above. Change-Id: I9873184c0df4a0b4455f803828e2719887e545db Signed-off-by: Michael Niewöhner <foss@mniewoehner.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/52210 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions