Very minor formatting changes

proxb / PoshRSJob

Provides an alternative to PSjobs with greater performance and less overhead to run commands in the background, freeing up the console and allowing throttling on the jobs.

MIT License

541 stars 87 forks source link

Very minor formatting changes #192

Open codykonior opened 6 years ago

codykonior commented 6 years ago

Changes proposed in this pull request:

Some files had BOM headers, these were removed so they're raw UTF8
Replaced the few instances of tabs with spaces
In the Pester tests if $PSVersion isn't defined then we define it so users can run Invoke-Pester

Has been tested on (remove any that don't apply):

Powershell 5.1
Windows 10

MVKozlov commented 6 years ago

Stripped BOM headers not supported in PSv2, It fail to detect utf8

codykonior commented 6 years ago

If this were true then the module would not function in the first place, because not every file used a BOM header. Only some files had BOM header and this standardises them.

Second of all the module still executes when run under powershell -version 2.

MVKozlov commented 6 years ago

May be this is becase it really does not contain any non-ascii symbols, so it non-utf ? :)

But contain BOM on accident, so the topic should sound like 'convert unneeded utf to ascii' ?