Skip to content

Fix approvers for analyzer.yml

Fabien Catteau requested to merge fix-codeowners-analyzer.yml into master

What does this MR do?

Fix the approvers list for analyzer.yml. Prior to this change, groupcomposition analysis is ignored, and groupstatic analysis wins. groupcomposition analysis should also be able to approve changes to analyzer.yml.

That's because analyzer.yml appears twice in the Code Owners file. See documentation:

When a file matches multiple entries in the CODEOWNERS file, the users from last pattern matching the file are displayed on the blob page of the given file. For example, you have the following CODEOWNERS file:

All approvers should be on same line, as documented in the example:

# Multiple codeowners can be specified, separated by spaces or tabs
CODEOWNERS @multiple @code @owners

Bug is illustrated by this screenshot taken from !93 (merged):

Capture_d_écran_2020-04-22_13.35.48

What are the relevant issue numbers?

None.

Does this MR meet the acceptance criteria?

Merge request reports