[Mlir-commits] [mlir] [mlir] List lead maintainers for MLIR (PR #146928)

[Oleksandr Alex Zinenko]

ftynse wrote:

> One issue here is that there wasn't a call for nomination for lead maintainer, there was form that (from memory) didn't have a box with areas (including "lead maintainer") but was asking something along the line of "which area do you want to maintain", and since the "tensor compiler" area is a recently well identified component that people stood up to maintain, I didn't feel like I was needed there, hence what I wrote! This is also why it feels to me like a "gotcha" and why I can't comprehend @ftynse 's answer here (unless I missed the checkbox in the form? In which case: my bad).
So there is the process as written in the policy, and then there is how it is put in practice. I don't want to fault the area team: it's hard to bootstrap things and I am certain that I wouldn't have been able to do a better job in this until opening this PR.
I wish the aftermath of opening the PR would have handled with more consideration instead of what I perceive as "hiding behind the process".

There was no such checkbox. I understand it may not have been specified clearly enough, but we were trying to keep the list as open as possible. The call was also to help us formulate the initial nominations, this does not preclude people from nominating themselves through PRs, but aggregates the discussion instead of having ~50 distinct and easy-to-miss PRs.

I am happy that we seem to have managed to clear out some of the points for you. If you have remaining questions, please don't hesitate to ask.

> To facilitate more focused and constructive discussion, would it make sense to split the nomination PRs so that each nomination can be evaluated independently? Or would the preference be to proceed with a single combined PR?

Why not both? :) We can start the discussion here, and if we can't converge before the Project Council meeting, we will our options (or ask them for their preferred way)?

https://github.com/llvm/llvm-project/pull/146928