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

Feature/java availability #106

Merged

Conversation

fredy-mederos
Copy link
Contributor

Including an utility class to easily obtain koin components and properties from java code.
There was also added some test files to validate the features.

…alidate injections in Java.

Including KoinJavaComponent utility object.
Including test cases to validate injections in Java.
@arnaudgiuliani arnaudgiuliani modified the milestones: 0.9.2, 1.0.0 Apr 9, 2018
@arnaudgiuliani
Copy link
Member

Hello, we will begin to check that for 1.0.0. Stay in touch

@arnaudgiuliani arnaudgiuliani mentioned this pull request May 2, 2018
21 tasks
@arnaudgiuliani arnaudgiuliani added type:feature-proposal core status:checking currently in analysis - discussion or need more detailed specs labels May 11, 2018
@arnaudgiuliani arnaudgiuliani added status:accepted accepted to be developed and removed status:checking currently in analysis - discussion or need more detailed specs labels Jun 16, 2018
@arnaudgiuliani arnaudgiuliani changed the base branch from 0.9.2 to 1.0.0 June 19, 2018 08:21
@arnaudgiuliani
Copy link
Member

Hello @fredy-mederos, I will prepare a dedicated koin-java module to isolate this static helper.
Stay tuned

@arnaudgiuliani
Copy link
Member

@fredy-mederos can you move your files into koin-java project?

@fredy-mederos
Copy link
Contributor Author

Yes I will do it in a few minutes.

@fredy-mederos
Copy link
Contributor Author

I will run the test cases. Wait for my confirmation to merge.

@fredy-mederos
Copy link
Contributor Author

All test passed!

@arnaudgiuliani arnaudgiuliani merged commit e0dfbcb into InsertKoinIO:1.0.0 Jun 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core status:accepted accepted to be developed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants