Describe the bug
Currently while wiring the classes we are instantiating dependencies.

However, this is something we should not do cause it moves the instantiation from dependencies inside the service provider and not anymore inside the definition.

The main difference here being that the definition is called when the actual is needing for instantiation and not when the whole provider needs to be registered.

To Reproduce
Use Xdebug

1. Go to any provider
2. Check when the dependency is instantiated
3. Removed the $this->getContainer()->get on the dependency
4. Check when the dependency is instantiated

Expected behavior
Developers should pass dependencies this way:

		$this->getContainer()->add( 'slack_subscriber', Subscriber::class )
			->addArgument( 'pagespeed_results_query' );

instead of:

		$this->getContainer()->add( 'slack_subscriber', Subscriber::class )
			->addArgument( $this->getContainer()->get( 'pagespeed_results_query' ) );

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

Acceptance Criteria (for WP Media team use only)
Clear instructions for developers, to be added before the grooming