Skip to content

Conversation

@decsny
Copy link
Member

@decsny decsny commented Nov 7, 2025

this is an initial pass of using file groups to help consolidate NXP areas and add more specific reviewers and also limiting "total reviewers" added per PR to NXP files if they are not relevant. more file groups will probably be added in the future especially in the drivers area.

based on #98884

nashif and others added 8 commits November 7, 2025 05:05
This new section allows defining a group of files in an area and makes
it possible to assign collaborators to the file group being defined.

The purpose of this new section is to allow fine tuning who is added as
reviewer when files change in a group. It is especially useful in large
areas with hundreds of files, for example platform areas.

Signed-off-by: Anas Nashif <[email protected]>
Deal with new section in the maintainer file defining file groups.

Signed-off-by: Anas Nashif <[email protected]>
Document file groups and how they should be used.

Signed-off-by: Anas Nashif <[email protected]>
Merge two workflows into one for code sharing an efficiency.

Signed-off-by: Anas Nashif <[email protected]>
Scripts only used by CI, so move it into that directory.

Signed-off-by: Anas Nashif <[email protected]>
Also request reviewes from maintainers of changes areas in the
maintainer file.

Signed-off-by: Anas Nashif <[email protected]>
File groups inherit file patterns from their parent area. A file will only
match a file group if it first matches the parent area's patterns, and then
also matches the file group's own patterns. This allows file groups to
further filter and subdivide files that are already covered by the area.

Signed-off-by: Anas Nashif <[email protected]>
Use the new file-groups feature to consolidate the NXP areas, and have
more specific collaborators/reviewers over certain areas.

Signed-off-by: Declan Snyder <[email protected]>
@decsny decsny requested review from dleach02 and nashif November 7, 2025 17:47
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 7, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants