Skip to content

Conversation

adietish
Copy link
Collaborator

@adietish adietish commented Sep 8, 2025

follows up on #162
depends on #165 which has to be merged first

@codecov-commenter
Copy link

codecov-commenter commented Sep 8, 2025

Codecov Report

❌ Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (71098f6) to head (91d2247).
⚠️ Report is 203 commits behind head on main.

Files with missing lines Patch % Lines
...view/steps/DevSpacesOpenShiftConnectionStepView.kt 0.00% 14 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##            main    #166    +/-   ##
======================================
  Coverage   0.00%   0.00%            
======================================
  Files          4      28    +24     
  Lines         26     995   +969     
  Branches       0     148   +148     
======================================
- Misses        26     995   +969     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adietish adietish force-pushed the allow-cancel--connecting branch from 6f4c481 to 1ca6c72 Compare September 8, 2025 14:48
@adietish adietish self-assigned this Sep 8, 2025
@adietish adietish moved this to Ready for Review in Eclipse Che Team C Backlog Sep 8, 2025

ProgressManager.getInstance().runProcessWithProgressSynchronously(
{
val indicator = ProgressManager.getInstance().progressIndicator
Copy link
Collaborator

Choose a reason for hiding this comment

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

variable indicator is unused

Signed-off-by: Andre Dietisheim <[email protected]>
@adietish adietish force-pushed the allow-cancel--connecting branch from 1ca6c72 to 91d2247 Compare September 8, 2025 20:27
@adietish
Copy link
Collaborator Author

adietish commented Sep 8, 2025

@msivasubramaniaan corrected, please re-review.

Copy link
Collaborator

@msivasubramaniaan msivasubramaniaan left a comment

Choose a reason for hiding this comment

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

LGTM

@msivasubramaniaan msivasubramaniaan merged commit 83198eb into redhat-developer:main Sep 9, 2025
5 checks passed
@adietish adietish deleted the allow-cancel--connecting branch September 9, 2025 08:48
@adietish adietish moved this from Ready for Review to ✅ Done in Eclipse Che Team C Backlog Sep 9, 2025
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.

3 participants