feat(core): throw an exception instead of logging due to module import misusage

[micalevisk]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Other... Please describe:

What is the current behavior?

if the imports prop of @Module() has any class annotated with @Controller(), @Catch() or @Injectable(), we'll get an warning message like the following:

[DependenciesScanner] In the next major version, Nest will not allow classes annotated with @Injectable(), @Catch(), and @Controller() decorators to appear in the "imports" array of a module.
Please remove "AppController" (including forwarded occurrences, if any) from all of the "imports" arrays.

Scope [AppModule]

What is the new behavior?

as this is not allowed in v9, the app now exits with an error:

Does this PR introduce a breaking change?

• Yes
• No

[micalevisk]

after this PR and PR #9591 there will be no more TODO(v9) comments on master 🥳

---

Other changes that I found that we should make on v9 are:

nest/packages/common/utils/shared.utils.ts

Lines 34 to 38 in 6f5986d

	/**
	* Deprecated. Use the "addLeadingSlash" function instead.
	* @deprecated
	*/
	export const validatePath = addLeadingSlash;

nest/packages/common/interfaces/nest-application.interface.ts

Lines 64 to 66 in 6f5986d

	/**
	* Starts the application (can be awaited).
	* @deprecated use "listen" instead.

nest/packages/common/interfaces/nest-application.interface.ts

Lines 141 to 143 in 6f5986d

	/**
	* Starts all connected microservices and can be awaited.
	* @deprecated use "startAllMicroservices" instead.

nest/packages/common/interfaces/nest-microservice.interface.ts

Lines 21 to 23 in 6f5986d

	/**
	* Starts the microservice (can be awaited).
	* @deprecated use "listen" instead.

[coveralls]

Pull Request Test Coverage Report for Build fa82382f-7730-40ef-9fee-18244c47f3d1

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.001%) to 94.106%

---

Totals
Change from base Build dc6b1e43-1d4d-4009-8b53-5ee2fd59bb11:	-0.001%
Covered Lines:	5780
Relevant Lines:	6142

---

💛 - Coveralls

[kamilmysliwiec]

Other changes that I found that we should make on v9 are:

Feel free to create PRs for those @micalevisk! 🙌
LGTM