hmlongco / Factory

A new approach to Container-Based Dependency Injection for Swift and SwiftUI.
MIT License
1.7k stars 107 forks source link

Name spacing conflict #161

Closed bigearsenal closed 8 months ago

bigearsenal commented 8 months ago

Hi there! Thank you for an awesome library. We faced a problem with the namespacing:

In summary, we can not use the name Container for new structure anymore, since it conflicts with Factory's Container and we can not use Factory.Container when there is already a struct Factory<T> in the library. (The compiler is confused between struct Factory<T> and module name Factory>.

I suggest renaming the Container to DIContainer

hmlongco commented 8 months ago

The current version has an alias for Container, FactoryContainer

bigearsenal commented 8 months ago

Thank for answering, I saw that but they are right now not publicly visible.

hmlongco commented 8 months ago

derp

bigearsenal commented 8 months ago

Saw the commit. Thank you!