Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: rename codeowners file #1667

Merged
merged 1 commit into from
Nov 27, 2024
Merged

Conversation

tkimura4
Copy link
Collaborator

Description

In the tier4/autoware.universe repository, we will no longer inherit the CODEOWNER settings from autowarefoundation/autoware.universe. Instead, reviewers will be manually assigned. To implement this change, we will rename the CODEOWNER file (rather than deleting it, to avoid conflicts during future merges).

Related links

(TIERIV Internal Link) https://star4.slack.com/archives/CTEJP8L4T/p1732674470656089

How was this PR tested?

In tier4/autoware.universe#1664, it was confirmed that reviewers are no longer automatically assigned due to this change.

Using the following command, it was also verified that the file's history has been preserved despite the rename
$ git log --name-status --follow .github/_CODEOWNERS

Notes for reviewers

None.

Interface changes

The automatic reviewer assignment via the CODEOWNERS file will no longer occur.

Effects on system behavior

None.

Signed-off-by: tomoya.kimura <[email protected]>
@tkimura4 tkimura4 requested a review from rej55 November 27, 2024 08:40
Copy link

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

Copy link
Collaborator

@rej55 rej55 left a comment

Choose a reason for hiding this comment

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

image

@rej55 rej55 merged commit 9536989 into awf-latest Nov 27, 2024
20 checks passed
@rej55 rej55 deleted the chore/rename_code_owner_file branch November 27, 2024 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants