New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

New Rule: Use Dependency Injection #140

Open

AlissonRS opened this issue Jan 10, 2024 · 1 comment

Labels

Accepting PRs or funding

AlissonRS commented Jan 10, 2024

Controllers and Services should not have properties, apart from the ones injected via DI (e.g. in the constructor).

There was a similar rule in this POC, I think it's a very nice rule: https://github.com/unlight/eslint-plugin-nestjs/blob/master/src/use-dependency-injection/use-dependency-injection.md

Owner

darraghoriordan commented Feb 3, 2024

Sounds good to me! Feel free to create a PR based on that the code from the other repo

darraghoriordan added the Accepting PRs or funding label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment