Skip to content

Comments

Someone touched my perfect code for WPFTweaksRemoveCopilot !#4068

Merged
ChrisTitusTech merged 1 commit intoChrisTitusTech:mainfrom
GabiNun:Update-WPFTweaksRemoveCopilot
Feb 18, 2026
Merged

Someone touched my perfect code for WPFTweaksRemoveCopilot !#4068
ChrisTitusTech merged 1 commit intoChrisTitusTech:mainfrom
GabiNun:Update-WPFTweaksRemoveCopilot

Conversation

@GabiNun
Copy link
Contributor

@GabiNun GabiNun commented Feb 17, 2026

Type of Change

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

Description

the if statment was incorrect and also isn't needed since who is removing that appx package? and it wont even show a error

Issue related to PR

  • Resolves #

@erffy
Copy link
Contributor

erffy commented Feb 17, 2026

Maybe someone among us is secretly using copilot and wants to ruin it. 😆

@GabiNun
Copy link
Contributor Author

GabiNun commented Feb 17, 2026

Maybe someone among us is secretly using copilot and wants to ruin it. 😆

lol

@ChrisTitusTech
Copy link
Owner

ChrisTitusTech commented Feb 17, 2026

I'm having copilot review this code... 😂

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR attempts to simplify the PowerShell script that removes Copilot-related packages by removing an if statement check and the wildcard from a Get-AppxPackage command. The author claims the if statement was incorrect and unnecessary. However, the changes introduce critical bugs that will cause the script to fail.

Changes:

  • Removed the if statement that checks if the MicrosoftWindows.Client.CoreAI package exists before attempting to remove it
  • Removed the wildcard from the Get-AppxPackage command for the CoreAI package
  • Left an orphaned closing brace that creates a syntax error

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ChrisTitusTech
Copy link
Owner

ROFL copilot says it fails when the manual unit test passes it. I think it doesn't like you, have been calling it a "clanker" in your off hours? @GabiNun

I'll merge this tomorrow.

@GabiNun
Copy link
Contributor Author

GabiNun commented Feb 17, 2026

ROFL copilot says it fails when the manual unit test passes it. I think it doesn't like you, have been calling it a "clanker" in your off hours? @GabiNun

I'll merge this tomorrow.

I have called ai a clanker more than once :)

@GabiNun GabiNun changed the title Someone touched my perfect code for WPFTweaksRemoveCopilot Someone touched my perfect code for WPFTweaksRemoveCopilot ! Feb 18, 2026
@github-actions github-actions bot added the bug Something isn't working label Feb 18, 2026
@ChrisTitusTech ChrisTitusTech merged commit 7fb1643 into ChrisTitusTech:main Feb 18, 2026
12 checks passed
@GabiNun GabiNun deleted the Update-WPFTweaksRemoveCopilot branch February 18, 2026 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants