-
Notifications
You must be signed in to change notification settings - Fork 528
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
: Refactor PortForwardService to Replace Anonymous Thread with Lambda #3300
Conversation
Eclipse JKube CI ReportStarted new GH workflow run for #3300 (2024-08-05T12:49:43Z) ⚙️ JKube E2E Tests (10248899841)
|
@Er-Sadiq : In order to contribute to Eclipse, you would need to create an Eclipse Account and Sign Eclipse Contributor Agreement. Once you've created an eclipse account you would need to update your commit message with a
|
…sue:#3285fixed issue:eclipse-jkube#3285 Signed-off-by: Sadiq Ahmed Killedar <[email protected]>
@rohanKanojia Hi , I have amended my commit and have signed the ECA . if all is good kindly accept my PR |
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx!
@Er-Sadiq : Please link issue number in your pull request description like this:
|
@rohanKanojia I've already mentioned it at the top |
Fixes #3285
Changes Made:
Replaced the anonymous inner class in the Runtime.getRuntime().addShutdownHook method with a lambda expression.