You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.
It seems like it would be @lx75249 because of [Integrate cpp package #5251] and it looks like @lx75249 gets asked to review in PRs to /cpp-package/ e.g. #9220 and #7495. Having a code owner is important PRs since some PRs can't be merged without having someone who is familiar with the package to review even if someone else actually reviews the code and all the code-owner has to do is write LGTM.
The text was updated successfully, but these errors were encountered:
Why isn't there a code owner for
/cpp-package/
in CODEOWNERS?It seems like it would be @lx75249 because of [Integrate cpp package #5251] and it looks like @lx75249 gets asked to review in PRs to
/cpp-package/
e.g. #9220 and #7495. Having a code owner is important PRs since some PRs can't be merged without having someone who is familiar with the package to review even if someone else actually reviews the code and all the code-owner has to do is write LGTM.The text was updated successfully, but these errors were encountered: