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

WFCORE-6440 HttpManagementAddHandler references ServerEnvironment incorrectly #5592

Merged
merged 2 commits into from
Aug 3, 2023

Conversation

pferraro
Copy link
Contributor

@pferraro pferraro requested a review from yersan July 31, 2023 13:28
Comment on lines +160 to +161
// TODO WFCORE-6321 Expose ServerEnvironment via a capability
// Supplier<ServerEnvironment> environment = builder.requiresCapability("org.wildfly.server.environment", ServerEnvironment.class);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Jul 31, 2023
@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Aug 2, 2023
@yersan yersan merged commit 8ba2478 into wildfly:main Aug 3, 2023
12 checks passed
@yersan
Copy link
Collaborator

yersan commented Aug 3, 2023

Thanks @pferraro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
2 participants