Skip to content

improve support for guice#65

Merged
brharrington merged 3 commits into
Netflix:masterfrom
brharrington:guice-module
Nov 14, 2014
Merged

improve support for guice#65
brharrington merged 3 commits into
Netflix:masterfrom
brharrington:guice-module

Conversation

@brharrington

Copy link
Copy Markdown
Contributor

Adds modules that can be used to easily inject
the registry with governator for running the
actual app or with a test module that creates
an isolated registry for unit tests.

If the GcLogger is started multiple times it
will now log instead of throw. That class needs
to be refactored, but that will be a separate
PR. For this update the focus is just on making
the module work.

Adds modules that can be used to easily inject
the registry with governator for running the
actual app or with a test module that creates
an isolated registry for unit tests.

If the GcLogger is started multiple times it
will now log instead of throw. That class needs
to be refactored, but that will be a separate
PR. For this update the focus is just on making
the module work.
@brharrington brharrington added this to the 1.0-RC1 milestone Nov 14, 2014
Updating the dependencies to reflect direct usage
of guice (and governator in tests). This was working
before due to transitive dependencies.
brharrington added a commit that referenced this pull request Nov 14, 2014
@brharrington brharrington merged commit ebcf154 into Netflix:master Nov 14, 2014
@brharrington brharrington mentioned this pull request Nov 15, 2014
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.

1 participant