Skip to content

Conversation

@nashif
Copy link
Member

@nashif nashif commented Oct 17, 2025

  • ci: assigner: merge maintainer check into assigner workflow
  • scripts: move set_assignee.py into scripts/ci
  • scripts: set_assignee: request review from maintainers of changed areas

@nashif nashif force-pushed the topic/maintainer/review_by_m branch from 3aecae2 to 4f55fcc Compare October 17, 2025 12:59
FLAGS+=" -M MAINTAINERS.yml"
if [ "${{ github.event_name }}" = "pull_request_target" ]; then
FLAGS+=" -P ${{ github.event.pull_request.number }} --updated-manifest pr_west.yml"
FLAGS+=" -P ${{ github.event.pull_request.number }} --updated-manifest pr_west.yml" --updated-mantainer-file pr_MAINTAINERS.yml"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably meant to be:

Suggested change
FLAGS+=" -P ${{ github.event.pull_request.number }} --updated-manifest pr_west.yml" --updated-mantainer-file pr_MAINTAINERS.yml"
FLAGS+=" -P ${{ github.event.pull_request.number }} --updated-manifest pr_west.yml --updated-mantainer-file pr_MAINTAINERS.yml"

?

Merge two workflows into one for code sharing an efficiency.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Scripts only used by CI, so move it into that directory.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Also request reviewes from maintainers of changes areas in the
maintainer file.

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
@nashif nashif force-pushed the topic/maintainer/review_by_m branch from 4f55fcc to 1f9469b Compare October 20, 2025 12:22
@sonarqubecloud
Copy link

@nashif
Copy link
Member Author

nashif commented Nov 5, 2025

superseded by #98884

@nashif nashif closed this Nov 5, 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