-
Notifications
You must be signed in to change notification settings - Fork 5
Allow to cancel connecting #166
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
Allow to cancel connecting #166
Conversation
Codecov Report❌ Patch coverage is
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. 🚀 New features to boost your workflow:
|
6f4c481
to
1ca6c72
Compare
|
||
ProgressManager.getInstance().runProcessWithProgressSynchronously( | ||
{ | ||
val indicator = ProgressManager.getInstance().progressIndicator |
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.
variable indicator
is unused
Signed-off-by: Andre Dietisheim <[email protected]>
1ca6c72
to
91d2247
Compare
@msivasubramaniaan corrected, please re-review. |
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
follows up on #162
depends on #165 which has to be merged first