Skip to content

update edge uninstall#4666

Open
GabiNun2 wants to merge 9 commits into
ChrisTitusTech:mainfrom
GabiNun2:update-edge-uninstall
Open

update edge uninstall#4666
GabiNun2 wants to merge 9 commits into
ChrisTitusTech:mainfrom
GabiNun2:update-edge-uninstall

Conversation

@GabiNun2

@GabiNun2 GabiNun2 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • UI/UX improvement

Description

added a if statment so it won't error out if edge isn't installed and updated the description and moved it stright into tweaks.json

@GabiNun2 GabiNun2 requested a review from ChrisTitusTech as a code owner June 8, 2026 14:35
@jnsh-rf

jnsh-rf commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

maybe shorten it a bit:

Uninstalls Microsoft Edge by creating dummy MicrosoftEdge.exe file in the legacy Edge folder. This tricks Windows into unlocking the official Edge uninstaller allowing for a system-level removal.

@GabiNun2

GabiNun2 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

maybe shorten it a bit:

Uninstalls Microsoft Edge by creating dummy MicrosoftEdge.exe file in the legacy Edge folder. This tricks Windows into unlocking the official Edge uninstaller allowing for a system-level removal.

sure

@ChrisTitusTech ChrisTitusTech added the Refactor For cleaning up and refactoring code to be more efficient. label Jun 8, 2026

@ChrisTitusTech ChrisTitusTech left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prefer not to mix ' and " in these invoke scripts.

rather escape the " example: " than a single '

@GabiNun2

Copy link
Copy Markdown
Contributor Author

prefer not to mix ' and " in these invoke scripts.

rather escape the " example: " than a single '

fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Refactor For cleaning up and refactoring code to be more efficient.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants