Skip to content

Comments

refactor(core-container): bind plugin options before registering the plugin#2375

Merged
spkjp merged 21 commits into2.4from
refactor/container-plugin-opts
Apr 9, 2019
Merged

refactor(core-container): bind plugin options before registering the plugin#2375
spkjp merged 21 commits into2.4from
refactor/container-plugin-opts

Conversation

@faustbrian
Copy link
Contributor

@faustbrian faustbrian commented Apr 8, 2019

Proposed changes

Resolves #2371 after #2364 is merged

To-Do

  • Adjust tests that rely on Shared.Config mocking

Types of changes

  • Refactoring (improve a current implementation without adding a new feature or fixing a bug)

Checklist

  • I have read the CONTRIBUTING documentation
  • Lint and unit tests pass locally with my changes

@ghost
Copy link

ghost commented Apr 8, 2019

The ci/circleci: test-node11-functional job is failing as of e03ede5c145bacc881a52e49b754f5691031b0b4. Please review the logs for more information.

Once you've pushed the fixes, the build will automatically re-run. Thanks!

@faustbrian faustbrian marked this pull request as ready for review April 9, 2019 07:48
@spkjp spkjp merged commit 1375ceb into 2.4 Apr 9, 2019
@ghost ghost deleted the refactor/container-plugin-opts branch April 9, 2019 13:17
@ghost ghost removed the Status: In Progress label Apr 9, 2019
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.

2 participants