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

Add constructor injected dependencies to the graph #53

Open
dvdciri opened this issue Aug 19, 2018 · 0 comments
Open

Add constructor injected dependencies to the graph #53

dvdciri opened this issue Aug 19, 2018 · 0 comments

Comments

@dvdciri
Copy link
Owner

dvdciri commented Aug 19, 2018

Daggraph need to be able to recognise constructor injected dependencies and add them to the graph.

There are some open questions here:

  • Where should we put the class which is constructor injecting some dependencies?
  • Where should we put the injected dependencies?

This is the regex for koltin: /(?:class)\s(\w*).*?(?:@Inject).*?(?:constructor)\((.*?)\)\s?\{/gs
This is the regex for java: /(?:@Inject).*?\s(\w*?)\((.*?)\)\s?{/gs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant