7 Feb
2025
7 Feb
'25
1:27 p.m.
Besides, since you mentioned uncertainty around apisets,
I mentioned it because it required a closer look, and I ran out of time that day. That said, I think the change is fine. It needed a closer look to review for this MR anyway.
I wonder if such refactoring would merit a separate MR.
Sure, I created !7287.
I think it would be good to settle this first. The whole push/pop approach adds extra effort that wouldn’t be necessary if we decide to go with it.
There are more:
In most cases, I’d say the exclusion shouldn’t apply. A few cases might be better handled as regular imports. Either way, I don’t think changing relay exclude is a big deal.