search

typestack / typeorm-typedi-extensions

Dependency injection and service container integration with TypeORM using TypeDI library.
MIT License
262 stars 37 forks source link

Renaming decorators #23

Closed MichalLytek closed 6 years ago

MichalLytek commented 6 years ago

I think that it would be better if we rename the @OrmRepository, @OrmManager and @OrmConnection decorators.

TypeDI has @Inject decorator so we should be consistent with this: @InjectRepository, @InjectManager and @InjectConnection 😉 Other libs has the same convention.

@pleerock What do you think about this?

pleerock commented 6 years ago

I don't like that they are longer then they were, but yeah it sounds logically correct. We can do that.

MichalLytek commented 6 years ago

Closed by #24

github-actions[bot] commented 3 years ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.