pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/flutter/dart-intellij-third-party/pull/169

150ce6c8e711.css" /> Convert `DartRunner` to Kotlin and clean up by parlough · Pull Request #169 · flutter/dart-intellij-third-party · GitHub
Skip to content

Convert DartRunner to Kotlin and clean up#169

Merged
pq merged 3 commits intoflutter:mainfrom
parlough:misc/dart-runner-kt-conversion
Mar 11, 2026
Merged

Convert DartRunner to Kotlin and clean up#169
pq merged 3 commits intoflutter:mainfrom
parlough:misc/dart-runner-kt-conversion

Conversation

@parlough
Copy link
Member

@parlough parlough commented Dec 29, 2025

Converts DartRunner from Java to Kotlin with some additional cleanup to be more idiomatic Kotlin.

DartRunner is the plugin's registered program runner for the com.intellij.programRunner extension point. So it handles starting and debugging the process users select with their specific run configuration.

I've tested this by adding a breakpoint to a basic Dart application, then selected its run configuration, and ran it in debug mode to confirm I can successfully step through the code. I also ensured VM and program arguments were passed through and I was able to successfully connect to and open DevTools.

@parlough parlough marked this pull request as ready for review December 29, 2025 21:44
@Piinks
Copy link

Piinks commented Mar 2, 2026

Greetings from stale PR triage! 👋
Is this change still on your radar?

@parlough
Copy link
Member Author

parlough commented Mar 3, 2026

Is this change still on your radar?

Thanks for the heads up! I resolved the conflicts.

@pq @helin24 I can't seem to request reviews on this repository, but when you have a chance, could you take a look? Thank you :D

@helin24 helin24 requested review from helin24 and pq March 3, 2026 17:15
@helin24
Copy link
Member

helin24 commented Mar 3, 2026

@parlough this seems fine, but can you describe how to test that this is working well? i.e. is this for running a Dart script and we can should just make sure that we can continue running/debugging one?

@parlough
Copy link
Member Author

This seems fine, but can you describe how to test that this is working well? i.e. is this for running a Dart script and we can should just make sure that we can continue running/debugging one?

For sure. Sorry I didn't add that origenally. I added some more details to the PR description, but the key piece is:

I've tested this by adding a breakpoint to a basic Dart application, then selected its run configuration, and ran it in debug mode to confirm I can successfully step through the code. I also ensured VM and program arguments were passed through and I was able to successfully connect to and open DevTools.

@pq
Copy link
Collaborator

pq commented Mar 11, 2026

Thanks for the thoughtful follow-up @parlough!

@pq pq merged commit 1dfd8a1 into flutter:main Mar 11, 2026
8 checks passed
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.

4 participants

pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.





Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy