[PR #342] Handle missing /target argument for build command. #1150

Closed
opened 2026-01-29 16:57:49 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/ElectronNET/Electron.NET/pull/342

State: closed
Merged: Yes


Ran into this myself today, if a user fails to provide the /target argument when running electronize build they simply get a KeyNotFoundException.

This change checks for the presence of this argument and, if not present, displays help information to the user as well as gracefully exiting.

Related to https://github.com/ElectronNET/Electron.NET/issues/296

As a sidenote, is there a reason we don't use a command line parsing library (for example - https://github.com/commandlineparser/commandline) to handle these cases for us?

**Original Pull Request:** https://github.com/ElectronNET/Electron.NET/pull/342 **State:** closed **Merged:** Yes --- Ran into this myself today, if a user fails to provide the `/target` argument when running `electronize build` they simply get a `KeyNotFoundException`. This change checks for the presence of this argument and, if not present, displays help information to the user as well as gracefully exiting. Related to https://github.com/ElectronNET/Electron.NET/issues/296 As a sidenote, is there a reason we don't use a command line parsing library (for example - https://github.com/commandlineparser/commandline) to handle these cases for us?
claunia added the pull-request label 2026-01-29 16:57:49 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/Electron.NET#1150