-
Notifications
You must be signed in to change notification settings - Fork 46
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
Overhaul core Tracker: extract IoC Container #1187
Labels
Comments
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
to josecelano/torrust-tracker
that referenced
this issue
Jan 17, 2025
josecelano
added a commit
that referenced
this issue
Jan 17, 2025
20018ab fix: [#1187] doc link error (Jose Celano) 3d0f4f8 refactor: [#1187] use AppContainer in test environments (Jose Celano) c45a12b refactor: [#1187] rename fn tracker_factory to initialize_tracker (Jose Celano) aa9f1c3 refactor: [#1187] move fn initialize_tracker_dependencies (Jose Celano) a4d8da0 refactor: [#1187] inline fn initialize_tracker (Jose Celano) 36db088 refactor: [#1187] inline fn initialize_globals_and_tracker (Jose Celano) 4aea9db refactor: [#1187] extract fn initialize_app_container (Jose Celano) 747b58d refactor: [#1187] extract one function and rename another one (Jose Celano) 8bea521 refactor: [#1187] extract IoC Container (Jose Celano) Pull request description: Overhaul core Tracker: extract IoC Container ACKs for top commit: josecelano: ACK 20018ab Tree-SHA512: 5bb7d6fc2f3e577547b1532b3267f3f1d7eddd938b883a6a3fb31a3a3bd0e3b8001325e3c6d45261da46c34fd2c4312b627abc0054efd7097e1721375b325b5f
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Parent issue: #1181
I'm working on a big refactor. I'm extracting responsibilities from the core tracker. The
Tracker
is acting like a kind of IoC container because it instantiates many services. Since I'm extracting those services from the tracker we need a container for them. I'm going to add a IoC container so this code:can be replaced with something like:
In the Index we called AppData but I think I will call it
AppContainer
.There is a crate shaku to handle compilation time dependencies but I don't want to introduce complex dependencies for now. And some dependencies are defined at runtime (database, ..).
The text was updated successfully, but these errors were encountered: