I don't think so that you use best practice.
First of all: Avoid async void methods There is a lof places with bad code when requests will be block.
First of all: Avoid async void methods There is a lof places with bad code when requests will be block.
Whilst there is some merit of what you had said, there would be some loose ends because the conversion to async was a significant change. I think the nop team did a good job and I'm happy about the direction. nop is opened sourced so you're welcome to change it if you're not satisified.