This is an archive of the discontinued LLVM Phabricator instance.

[NFC][clang-tools-extra] Inclusive language: replace master with main
ClosedPublic

Authored by quinnp on Nov 17 2021, 9:09 AM.

Details

Summary

[NFC] As part of using inclusive language within the llvm project, this patch
replaces master with main in SubModule2.h.

Diff Detail

Event Timeline

quinnp created this revision.Nov 17 2021, 9:09 AM
quinnp requested review of this revision.Nov 17 2021, 9:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 17 2021, 9:09 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
sammccall added inline comments.Nov 17 2021, 10:23 AM
clang-tools-extra/clangd/Protocol.h
141

This is quoting https://microsoft.github.io/language-server-protocol/specification.
I think they accept PRs though.

quinnp marked an inline comment as done.Nov 17 2021, 10:38 AM
quinnp added inline comments.
clang-tools-extra/clangd/Protocol.h
141

Thanks @sammccall, I'll remove this change from the patch.

quinnp updated this revision to Diff 387990.Nov 17 2021, 10:40 AM
quinnp marked an inline comment as done.

Addressing review comments. Reverting change to Protocol.h.

quinnp edited the summary of this revision. (Show Details)Nov 17 2021, 10:41 AM

@sammccall could you take a look at the other changes when you have time? Thanks!

sammccall accepted this revision.Nov 25 2021, 7:46 AM

Sorry, LG!
(I'm not very familiar with clang-modularize, but this seems to be the only place the term is used)

This revision is now accepted and ready to land.Nov 25 2021, 7:46 AM