chore: add Sam into team + order member list#187
Conversation
Signed-off-by: Wen Zhou <wenzhou@redhat.com>
📝 WalkthroughWalkthroughThe Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
The OWNERS file contains untrusted users, which makes it INVALID. The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the opendatahub-io org. You can then trigger verification by writing
|
|
Cannot approve the pull request: Error: openshift-ci[bot] is not included in the approvers role in the OWNERS file |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@OWNERS_ALIASES`:
- Around line 2-13: The llm-d-team member list in OWNERS_ALIASES is not sorted;
reorder the entries under the llm-d-team key into case-insensitive alphabetical
order (compare usernames lowercased) so the list matches the PR title claim to
"order member list", and also update the PR or file to clarify which username
corresponds to "Sam" (or add the correct username for Sam) — check the current
entries aneeshkp, anishasthana, acardace, dagrayvid, evacchi, Gregory-Pereira,
hexfusion, nainaz, RishabhSaini, zdtsw, wseaton and reorder them and either
annotate or replace the appropriate entry to explicitly represent Sam.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Central YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro
Run ID: d31a2ad9-d5dd-4f1f-9d44-b9d610b74bc1
📒 Files selected for processing (1)
OWNERS_ALIASES
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Gregory-Pereira, zdtsw The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Cannot approve the pull request: Error: openshift-ci[bot] is not included in the approvers role in the OWNERS file |
|
/lgtm Thanks |
|
Cannot apply the lgtm label because Error: hexfusion is not included in the reviewers role in the OWNERS file |
|
/override ci/prow/images |
|
@zdtsw: Overrode contexts on behalf of zdtsw: ci/prow/images DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/override ci/prow/unit |
|
@zdtsw: Overrode contexts on behalf of zdtsw: ci/prow/unit DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Description
cc @anishasthana @hexfusion
How Has This Been Tested?
Merge criteria:
Summary by CodeRabbit