One of my really big pet peeves is seeing a dependency injection container used as a service locator. Where constructors have a ContainerInterface argument and get services out of it.
Even worse, the container allows global access and is used as a giant global for your dependencies. Accessed anywhere you might wanna use something.
