-
Notifications
You must be signed in to change notification settings - Fork 755
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
[Debugger] Add support to launch programs in fast-run mode #43709
[Debugger] Add support to launch programs in fast-run mode #43709
Conversation
a0875af
to
ec13a1d
Compare
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.
Copilot reviewed 5 out of 12 changed files in this pull request and generated no comments.
Files not reviewed (7)
- misc/debug-adapter/modules/debug-adapter-core/spotbugs-exclude.xml: Language not supported
- language-server/modules/langserver-commons/src/main/java/org/ballerinalang/langserver/commons/workspace/RunContext.java: Evaluated as low risk
- language-server/modules/langserver-commons/src/main/java/org/ballerinalang/langserver/commons/workspace/WorkspaceManager.java: Evaluated as low risk
- language-server/modules/langserver-core/src/main/java/org/ballerinalang/langserver/command/executors/RunExecutor.java: Evaluated as low risk
- tests/jballerina-debugger-integration-test/src/test/java/org/ballerinalang/debugger/test/adapter/DebugOutputTest.java: Evaluated as low risk
- misc/debug-adapter/modules/debug-adapter-core/src/main/java/org/ballerinalang/debugadapter/DebugOutputLogger.java: Evaluated as low risk
- language-server/modules/langserver-core/src/main/java/org/ballerinalang/langserver/workspace/BallerinaWorkspaceManager.java: Evaluated as low risk
Comments suppressed due to low confidence (1)
misc/debug-adapter/modules/debug-adapter-core/src/main/java/org/ballerinalang/debugadapter/JBallerinaDebugServer.java:299
- The parameter name portName should be renamed to port for consistency.
private void attachToRemoteVM(String hostName, int portName) throws IOException,
...gserver-commons/src/main/java/org/ballerinalang/langserver/commons/workspace/RunContext.java
Show resolved
Hide resolved
...angserver-core/src/main/java/org/ballerinalang/langserver/command/executors/RunExecutor.java
Outdated
Show resolved
Hide resolved
...-adapter-core/src/main/java/org/ballerinalang/debugadapter/BallerinaExtendedDebugServer.java
Outdated
Show resolved
Hide resolved
44dccf9
to
f0008ac
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #43709 +/- ##
============================================
- Coverage 77.40% 77.39% -0.01%
- Complexity 58600 58622 +22
============================================
Files 3446 3447 +1
Lines 219114 219242 +128
Branches 28965 28971 +6
============================================
+ Hits 169604 169693 +89
- Misses 40080 40118 +38
- Partials 9430 9431 +1 ☔ View full report in Codecov by Sentry. |
41cde72
into
ballerina-platform:master
Purpose
$subject.
Resolves #43702.
Samples
Check List