mirror of
https://github.com/ElectronNET/Electron.NET.git
synced 2026-02-03 21:25:13 +00:00
Implement Migration Checks #1016
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @softworkz on GitHub (Nov 20, 2025).
Before implementing anything, we should build a list of checks that we want to perform on build automatically to prevent users who are migrating from previous versions, from running into issues due to incorrect project setup.
1. No package.json must be present in the project
Check
The project must not contain any file named package.json (neither package-lock.json)
Help Message
All properties from an existing package.json file must be specified as MSBuild properties in the project now.
(Link to Wiki docs)
Exception
The only exception is a package.json (and package-lock.json) in a folder named ElectronHostHook.
TODO: We may need to validate the contents of this package.json alongside any tsconfig.json if present
2. No electron-manifest.json must be present in the project
Check
The project must not contain any file named electron-manifest.json
Help Message
(Link to Wiki docs)
./Propertiesfolder(Link to Wiki docs)
3. Single electron-builder.json
Check
The project must contain a single electron-builder.json inside the
./PropertiesfolderHelp Message
./Propertiesfolder4. No parent-paths in electron-builder.json
Check
The electron-builder.json file must not contain any parent-path references
Help Message
5. User has the right kind of pubxml content
Check
Check for
WebPublishMethodandProjectGuidproperties in pubxml files. If present, then it's for ASP.Net, otherwise for all other app types.Then check the project type and whether the pubxml file (or all existing ones) are matching the project type.
Help Message
@softworkz commented on GitHub (Nov 20, 2025):
Note
Thinking about it: using npm for web/client related packages must still be possible, so that part needs consideration.
@softworkz commented on GitHub (Dec 7, 2025):
Added 5.