Add attempted code owner for MUON - #2333
Conversation
|
@ihrivnac @aphecetche I guess we agree on this. |
|
@ihrivnac unless you strongly object, I would try to merge this and similar PRs without having to wait for the full list of interested parties, because otherwise I get spammed as "fall back solution". |
ihrivnac
left a comment
There was a problem hiding this comment.
Hi,
There is no doubt about adding Laurent here, but I'd prefer to avoid multiple PR's for updating this file. To avoid you being spammed as "fall back solution", we can modify the file and inactivate all lines below the first one, which directs everything to "core".
Would this work for you?
|
Apart from the fact that I do want to be notified as "core" if some of the top-level files change (e.g. the travis and codecov stuff), removing core means that people will be able to add new files and folders at top-level, which for sure we want to catch. |
|
What is the issue with the incremental approach, out of curiosity? |
|
I did not propose to remove core. |
|
Can't we have a JIRA ticket for that? |
|
The JIRA ticket already exists (was create at the same time as the WIP PR): |
|
Anyways, as you prefer. If it's matter of a few days I guess I can survive with a few extra emails. |
|
This is now part of #2357. I guess we can close this. |
* [PWGEM-10] Add EMCal cluster cuts to new CutLibrary - Add the EMCal cluster cuts to new CutLibrary started by Daiki (PR AliceO2Group#2307) * [PWGEM-10] Fix MegaLinter Errors for PR AliceO2Group#2333
No description provided.