That should be the point of Bitcoin Core, remain the Core, stick to the basics, and do not add any complexity in the name of innovation.
By that metric, that's what the PR does. It removes more complex code than it adds.
I don't understand why everyone is flaming us for someone who doesn't frequently contribute to Core opening a PR advocating for a change they'd like to see. Just because there is a PR doesn't mean that it's a good idea or that will be merged.
Well even if you don't use Core you are still going to suffer the impact of whatever Core does. Say im running Knots now as my full node, im still going to complain about things Core does that I think are not good for BTC, because Core is a main part of the network as it stands, so even if I run Knots, if Core screws up along the way, the price is the same for everyone involved in running nodes irrespective of what software you are running. As far as removing lines of code, im not an expert to start talking about the code itself, but I understand game theory. If we remove all code that was added with segwit and beyond and just put 128MB blocks or whatever, you would have simpler code, but a bloated blockchain the moment someone wants to exploit it.