Skip to content
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.

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

Dependency injection #195

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

PhenX
Copy link

@PhenX PhenX commented Mar 12, 2024

With this, we can customize all dependencies :

  • ITemplateService
  • IMetadataReaderFactory
  • ITypeService
  • ITypeDependencyService
  • ITsContentGenerator
  • ITsContentParser
  • IFileSystem
  • IInternalStorage
  • Generator
  • ILogger (the one from the lib, not from Microsoft.Extensions.Logging)

@PhenX
Copy link
Author

PhenX commented Jun 3, 2024

Hello @jburzynski, before this pull request I forgot to ask if it was something you'd like to add in your library. Personally, I think it would add a lot of flexibility, nearly everything would be customizable (except logging, for now). What do you think?

@PhenX
Copy link
Author

PhenX commented Sep 5, 2024

Hello @jburzynski, any news about this pull request ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant