Update codeowners and CI config for AOT #28604
Conversation
srnagar
requested review from
AlexGhiondea,
JonathanGiles,
anuchandy,
conniey,
hallipr,
joshfree,
jsquire,
lmolkova and
pallavit
as code owners
|
API change check for API changes are not detected in this pull request for |
|
API change check for API changes are not detected in this pull request for |
JimSuplizio
left a comment
JimSuplizio
left a comment
There was a problem hiding this comment.
@srnagar this may or may not actually appease the MSDocs. Either way, having the readme instead of a pure empty javadoc is the right way to go.
|
|
||
| # Git Hub integration and bot rules | ||
| /.github/ @AlexGhiondea @jsquire | ||
| /.github/ @AlexGhiondea @jsquire |
There was a problem hiding this comment.
@AlexGhiondea, @jsquire, do you want to scope this to the fabricbot.json file so you aren't pinged for review when CODEOWNERs change?
There was a problem hiding this comment.
@alzimmermsft: Please do not change this. We need visibility on CODEOWNERS changes in order to keep the bot rules up to date with "Service Attention" contacts or when paths for PR labels change.
There was a problem hiding this comment.
Got it, didn't realize it was tied closely to CODEOWNERS
There was a problem hiding this comment.
Unfortunately, it is - and its currently a manual sync due to bot limitations.
5 participants
This PR updates the codeowners for
sdk/aot/directory, includes perf and samples packages as additional modules and creates javadoc jar with README.