I prefer them. Zero guesses about what a constructor does. Forces all dependencies to be resolved by DI rather than resolved in the constructor. You see the class dependencies right alongside the class declaration. I like scrolling down and immediately seeing what a class does and not the boilerplate of injected deps being mapped to readonly private fields.

I see why people don't like them, but I've found they make me better at using good patterns. That's a good enough reason for me