Sycnex / Windows10Debloater

Script to remove Windows 10 bloatware.
MIT License
18.13k stars 2.04k forks source link

Bug: Param heading is wrong #8

Closed Skatterbrainz closed 6 years ago

Skatterbrainz commented 6 years ago

The Windows10Debloater.ps1 script appears to be missing a final closing "}" on the last function.

Sycnex commented 6 years ago

Hi Skatterbrainz,

I cannot seem to reproduce this regardless of what I run. Can you please provide a screenshot if it is continuing to still happen?

Sycnex commented 6 years ago

Any update on this? Otherwise if it works can I close this issue?

Skatterbrainz commented 6 years ago

The issue is with having three param() blocks, it throws an error. It should be one param() block with the [switch] params nested.

param([switch]$Debloat)

param([switch]$SysPrep)

param([switch]$StopEdgePDF)

should be...

param (

  [switch]$Debloat,

  [switch]$SysPrep,

  [switch]$StopEdgePDF

)

once I change that, everything appears to work fine.

Skatterbrainz commented 6 years ago

Any update on this?

Sycnex commented 6 years ago

Hi,

I haven't gotten a chance due to some personal life things going on. I will update it today if I get the chance.

On Mon, Mar 12, 2018, 12:29 PM Skatterbrainz notifications@github.com wrote:

Any update on this?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Sycnex/Windows10Debloater/issues/8#issuecomment-372434585, or mute the thread https://github.com/notifications/unsubscribe-auth/AHCc4HuNqJro58-zaOj7q199gCmb3Vn6ks5tdsyYgaJpZM4SPsrx .

Sycnex commented 6 years ago

This has been fixed as specified.