It's quite sobering realising a lot of shit code you see in mm exists because we just took it without _ANY_ review tags.
Joke.
IOW - no sign off. Review may have happened up to a point, but it stopped.
And that was taken as a 'merge this' signal.
Unreal.
@ljs How is that? Shouldn't some maintainer sign it off and send it to Linus on a PR?
@ljs one weird trick to become a kernel developer
@ffmancera not if your maintainer is just merging anything.
@ffmancera in mm we have real maintainers and the guy who is taking everything and sending to Linus, they're not the same.
@kdave sigh. But no tags required.
Review is 'people replied until they stopped replying at which point I just take it'.
Yeah things have got to change...
@kdave btw you can REALLY TELL who was relying on this prior to the great 'mm now actually has maintainers' change.
And there's tantrums...