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

Dependency Injection: Diagnostic for @Inject with more than one constructor. #174

Closed
3 tasks
Tracked by #16
ananya2801 opened this issue Oct 6, 2021 · 0 comments · Fixed by #200
Closed
3 tasks
Tracked by #16

Dependency Injection: Diagnostic for @Inject with more than one constructor. #174

ananya2801 opened this issue Oct 6, 2021 · 0 comments · Fixed by #200
Assignees
@ananya2801
Labels
diagnostic Diagnostic item

Comments

@ananya2801
Copy link
Contributor

ananya2801 commented Oct 6, 2021 •
edited
Loading

Spec

Check:

  • @Inject can apply to at most one constructor per class.

Potential Solution:

  • Ask the user which constructor they want to use @Inject with.
  • Remove @Inject annotation from the other constructor(s).

Related to #153
@kathrynkodama kathrynkodama added the diagnostic Diagnostic item label Oct 6, 2021
@ananya2801 ananya2801 mentioned this issue Oct 29, 2021
Merged
kathrynkodama pushed a commit that referenced this issue Nov 8, 2021
@ananya2801
Dependency Injection diagnostic and quick fix for #174 Inject with mu…
54bbb1a 
…ltiple constructors (#200)

* multiple injected constructors

* error appears for all constructors

* fix formatting, remove license

* fix more formatting issues
@ananya2801 ananya2801 mentioned this issue Nov 8, 2021
Closed
@kathrynkodama kathrynkodama mentioned this issue Jan 14, 2022
Open
83 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ananya2801 ananya2801

Labels
diagnostic Diagnostic item
Projects
None yet
Milestone
No milestone
2 participants
@kathrynkodama @ananya2801