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

issue #138: fix compilation error when using injection within an inne… #139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Luolc
Copy link

@Luolc Luolc commented Nov 23, 2016

…r class

I have read the CI test and the diff is just the order of generated methods in Henson class.

@stephanenicolas
Copy link
Collaborator

@Luolc let's move the discussion related to your PR here !

Yes we need the PR to be green. It's not about sorting the test methods (as tests methods should be orthogonal = independent from each other).

Here it's not the issue, or it's not phrased correctly: yes, you must swap gotoTest and gotoTestOne. If the order of these methods is random, ideally your PR should impose an order on the generated methods (but that could be a different issue).

@Luolc
Copy link
Author

Luolc commented Jan 4, 2017

@stephanenicolas Get! I will try to fix the orders problem :) Thx for help!

@stephanenicolas
Copy link
Collaborator

Np, and feel free to pressure us a bit to review the PR, we don't actively work on Dart 2 as if perfectly fits our need and we are quite busy with Toothpick, our new lib (and our other job tasks accessorily ;) ).

@Luolc
Copy link
Author

Luolc commented Jan 4, 2017

@stephanenicolas hah I will mention you once I update the PR. Good luck for your new task :)

@Luolc
Copy link
Author

Luolc commented Jan 4, 2017

@stephanenicolas green now :D plz have a check :)

@dlemures
Copy link
Collaborator

The PR is targeting an older version of Dart. @Luolc can you reproduce the issue you are trying to solve with the version 3.0.1?

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.

None yet

3 participants