Skip to content

[bolt] Move CODE_OWNERS.txt to Maintainers.txt #118082

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

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

DavidSpickett
Copy link
Collaborator

To align with: https://llvm.org/docs/DeveloperPolicy.html#maintainers

I have not changed the format of the file, my only goal here is that the project have a bolt/Maintainers.* so it is easy to find.

To align with: https://llvm.org/docs/DeveloperPolicy.html#maintainers

I have not changed the format of the file, my only goal
here is that the project have a `bolt/Maintainers.*` so it
is easy to find.
@llvmbot
Copy link
Member

llvmbot commented Nov 29, 2024

@llvm/pr-subscribers-bolt

Author: David Spickett (DavidSpickett)

Changes

To align with: https://llvm.org/docs/DeveloperPolicy.html#maintainers

I have not changed the format of the file, my only goal here is that the project have a bolt/Maintainers.* so it is easy to find.


Full diff: https://github.com/llvm/llvm-project/pull/118082.diff

2 Files Affected:

  • (renamed) bolt/Maintainers.txt ()
  • (modified) llvm/Maintainers.md (+1-1)
diff --git a/bolt/CODE_OWNERS.TXT b/bolt/Maintainers.txt
similarity index 100%
rename from bolt/CODE_OWNERS.TXT
rename to bolt/Maintainers.txt
diff --git a/llvm/Maintainers.md b/llvm/Maintainers.md
index 9744aa1aa43be9..4daf778f4760f1 100644
--- a/llvm/Maintainers.md
+++ b/llvm/Maintainers.md
@@ -424,7 +424,7 @@ [email protected] (email), [gkistanova](https://github.com/gkistanova) (GitHu
 Some subprojects maintain their own list of per-component maintainers.
 Others only have a lead maintainer listed here.
 
-[Bolt maintainers](https://github.com/llvm/llvm-project/blob/main/bolt/CODE_OWNERS.TXT)
+[Bolt maintainers](https://github.com/llvm/llvm-project/blob/main/bolt/Maintainers.txt)
 
 [Clang maintainers](https://github.com/llvm/llvm-project/blob/main/clang/Maintainers.rst)
 

@DavidSpickett
Copy link
Collaborator Author

Note that this file is not used by GitHub to assign reviewers, as explained here -

# Note that GitHub's concept of "code owner" is independent from LLVM's own

So moving it doesn't break anything.

(I will remove that note once all projects have moved to Maintainers.*)

@DavidSpickett DavidSpickett merged commit 085e7d2 into llvm:main Dec 2, 2024
10 checks passed
@DavidSpickett DavidSpickett deleted the bolt-maintainers branch December 2, 2024 09:13
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.

3 participants