dotnet electronize start should abort if build fails #68

Closed
opened 2026-01-29 16:29:47 +00:00 by claunia · 3 comments
Owner

Originally created by @yoDon on GitHub (Nov 12, 2017).

Originally assigned to: @robertmuehsig on GitHub.

Hi guys, at least in VS Code if there is a compiler error during the dotnet electronize start's build step, there are still enough old built *.exe's lying around that the electron app will appear to launch and run, but it's running the last successful build not the current code. Definitely a recipe for burning a ton of extra time debugging the wrong exe, and trying to remember to either manually clean bin and obj before each build or manually doing an explicit build step (which spews less into the terminal window) before running start. Having start abort if the build fails would be a huge help.

Originally created by @yoDon on GitHub (Nov 12, 2017). Originally assigned to: @robertmuehsig on GitHub. Hi guys, at least in VS Code if there is a compiler error during the dotnet electronize start's build step, there are still enough old built *.exe's lying around that the electron app will appear to launch and run, but it's running the last successful build not the current code. Definitely a recipe for burning a ton of extra time debugging the wrong exe, and trying to remember to either manually clean bin and obj before each build or manually doing an explicit build step (which spews less into the terminal window) before running start. Having start abort if the build fails would be a huge help.
claunia added the enhancement label 2026-01-29 16:29:47 +00:00
Author
Owner

@robertmuehsig commented on GitHub (Nov 15, 2017):

The build/start command will now stop if the publish command fails (e.g. compilation error occured) - maybe not perfect, but should be better then now.

@robertmuehsig commented on GitHub (Nov 15, 2017): The build/start command will now stop if the publish command fails (e.g. compilation error occured) - maybe not perfect, but should be better then now.
Author
Owner

@robertmuehsig commented on GitHub (Nov 15, 2017):

mhh... i broke it...

@robertmuehsig commented on GitHub (Nov 15, 2017): mhh... i broke it...
Author
Owner

@robertmuehsig commented on GitHub (Nov 15, 2017):

Not perfect, but could work. The code now searchs for "error". It would be way cleaner to get the exit code of the actual cmd.exe/bash, but I couldn't get it and the actual cmd.exe runs fine...

@robertmuehsig commented on GitHub (Nov 15, 2017): Not perfect, but could work. The code now searchs for "error". It would be way cleaner to get the exit code of the actual cmd.exe/bash, but I couldn't get it and the actual cmd.exe runs fine...
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/Electron.NET#68