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

Issue #3336 - $healthcheck warning when deployment update required #3394

Merged
merged 1 commit into from
Feb 28, 2022

Conversation

tbieste
Copy link
Contributor

@tbieste tbieste commented Feb 24, 2022

Issue #3336 - $healthcheck warning when deployment update required

Signed-off-by: Troy Biesterfeld tbieste@us.ibm.com

@tbieste tbieste self-assigned this Feb 24, 2022
@tbieste tbieste changed the title Issue #3336 - Add warning to $healcheck when deployment update required Issue #3336 - Add warning to $healthcheck when deployment update required Feb 24, 2022
@tbieste tbieste changed the title Issue #3336 - Add warning to $healthcheck when deployment update required Issue #3336 - $healthcheck warning when deployment update required Feb 24, 2022
Signed-off-by: Troy Biesterfeld <tbieste@us.ibm.com>
@@ -223,7 +223,7 @@

// Enable use of legacy whole-system search parameters for the search request
private final boolean legacyWholeSystemSearchParamsEnabled;

Copy link
Collaborator

Choose a reason for hiding this comment

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

Whitespace cleanup may cause merge issues with currently active PRs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Eclipse automatically cleans up the whitespace, so it was either do it now or later. I thought now would be fine.

@tbieste tbieste marked this pull request as ready for review February 28, 2022 16:20
Copy link
Member

@lmsurpre lmsurpre left a comment

Choose a reason for hiding this comment

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

LGTM

@tbieste tbieste merged commit 14bbbf5 into main Feb 28, 2022
@tbieste tbieste deleted the tbieste-issue-3336 branch February 28, 2022 18:21
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.

None yet

3 participants