Skip to content

OWNERS_ALIASES is not working with current automation#296

Merged
nirrozenbaum merged 1 commit intollm-d:mainfrom
nirrozenbaum:owners
Aug 18, 2025
Merged

OWNERS_ALIASES is not working with current automation#296
nirrozenbaum merged 1 commit intollm-d:mainfrom
nirrozenbaum:owners

Conversation

@nirrozenbaum
Copy link
Collaborator

current automation of the new workflow to automation owners lgtm and approval doesn't work with OWNERS_ALIASES.
revert back to use OWNERS file only.

@elevran
Copy link
Collaborator

elevran commented Aug 18, 2025

seems there's no reference to aliasing in the action's repo (e.g., https://github.com/jpmcb/prow-github-actions/blob/1accfe4a8ca80422f59da41949e6ea810a7b647d/src/utils/auth.ts#L226)

@nirrozenbaum nirrozenbaum enabled auto-merge (squash) August 18, 2025 07:15
@nirrozenbaum nirrozenbaum disabled auto-merge August 18, 2025 07:15
Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>
@nirrozenbaum nirrozenbaum merged commit ed78cd8 into llm-d:main Aug 18, 2025
5 checks passed
@nirrozenbaum nirrozenbaum deleted the owners branch August 18, 2025 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants