michael-hoke commented on code in PR #1403:
URL: https://github.com/apache/daffodil-vscode/pull/1403#discussion_r2344704198
##########
src/launchWizard/launchWizard.ts:
##########
@@ -510,11 +518,11 @@ class LaunchWizard {
Review Comment:
If I have a launch config with 'none' set, and I copy it using the Launch
Wizard, then change 'none' to 'execute' in the copy, the defaults are empty
strings. If we set the default values here ("Default Test Case" and
"${AskForValidatedTDMLPath}"), they'll get picked up on the switch.
I got here by looking at a change made in this PR in script.js where the
Path value is set to an empty string. I suspect that should probably be updated
as well, but I don't know exactly how to make that empty string show up.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
