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.

Sign up for GitHub

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

Jump to bottom

Load dependencies in order via dependency tree analysis #1

Closed
3 tasks done
BrainBacon opened this issue Sep 22, 2015 · 0 comments · Fixed by #3
Closed
3 tasks done

Load dependencies in order via dependency tree analysis #1

BrainBacon opened this issue Sep 22, 2015 · 0 comments · Fixed by #3
Assignees
@BrainBacon
Labels
enhancement
Milestone
0.2.0

Comments

@BrainBacon
Copy link
Owner

Right now nodep requires that dependencies are manually loaded in order. Nodep should analyze the injectables in each dependency and decide to load them accordingly. This will require a graceful failure when circular references are found.

  • implementation
  • documentation
  • testing
@BrainBacon BrainBacon added this to the 0.2.0 milestone Sep 22, 2015
@BrainBacon BrainBacon mentioned this issue Sep 22, 2015
Closed
4 tasks
@BrainBacon BrainBacon self-assigned this Sep 22, 2015
BrainBacon added a commit that referenced this issue Nov 10, 2015
@BrainBacon
feat(nodep.js): add dependency tree analysis
fcba22a 
Closes #1

BREAKING CHANGE: $p.load has now moved to $p.init
@BrainBacon BrainBacon mentioned this issue Nov 11, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BrainBacon BrainBacon

Labels
enhancement
Projects
None yet
Milestone
0.2.0
Development

Successfully merging a pull request may close this issue.

0.2.0 BrainBacon/Nodep
1 participant
@BrainBacon