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

Dart 1 support #3

Closed
maxwellpeterson-wf opened this issue Sep 30, 2018 · 3 comments
Closed

Dart 1 support #3

maxwellpeterson-wf opened this issue Sep 30, 2018 · 3 comments

Comments

@maxwellpeterson-wf
Copy link

Thank you for creating this great library! The organization I work for has been looking for DI solutions in Dart, but have been disappointed by the existing options due to the reasons you outlined in your Medium post.

We have a lot of Dart code and as a result we are still at least a few months away from transitioning our entire codebase to Dart 2. I notice the README states this library requires Dart 2, but I was wondering if there were any limitations in Dart 1 that prevent this from working properly?

It seems that if I add the missing new keywords, the rest of the syntax passes the analyzer just fine. Is there anything else we should be concerned about? If not I'm thinking we might maintain a fork of this that works on Dart 1 until we have finished our codebase migration to Dart 2, at which point we will switch back to the mainline.

Thanks for the great work here!

@letsar
Copy link
Contributor

letsar commented Oct 1, 2018

Hi @maxwellpeterson-wf,

Thank you for your interest in this library 😊 .

Since I didn't test this library with Dart 1, I don't know if there are limitations (beside new keyword), but I would be extremely careful about types.
I don't remember the details, but I know that I had issues in Dart 1 when I wanted to store an object in a Map<Type, Object> through a generic method, like in the kiwi container.

The fork is a great idea, let me know if you have any trouble, I would be happy to help if I can.
I'm also interested by any feedback you can give to make this library better, so don't hesitate 😃 !

Thank you.

@vanlooverenkoen
Copy link
Collaborator

@maxwellpeterson-wf because Dart 2 offers a lot of good features. With nullability coming in the future I think we should focus only on Dart 2. If there still is a big interest in Dart 1, please let me know so we can add it to the roadmap. For now I will close this issue

@maxwellpeterson-wf
Copy link
Author

@vanlooverenkoen sounds good, we have switched to Dart 2.

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

No branches or pull requests

3 participants