Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct issues with ConvertFrom-VDF and ConvertTo-VDF in SteamTools.psm1 #17

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

ciphernemo
Copy link

To address issue #12 . Correctly generate matches with a new regular expression in CovnertFrom-VDF and a re-written CovnertTo-VDF, which maintains object member order and removes extra quote marks.

Use LogTools module instead of Write-Host
Use LogTools module instead of Write-Host
Fixes for finding Steam and Steam Libraries, and checking to make sure local install (not owned) folder exists. Addressing $null on right side of comparison issue.
Removed my debugging shortcut for Set-Location
Added $PSScriptRoot\ instead of .\ when importing modules
Added $PSScriptRoot\ instead of .\ when importing modules
Added $PSScriptRoot\ instead of .\ when importing modules
Added $PSScriptRoot\ instead of .\ when importing modules
Use native String instead of System.String
Use native String instead of System.String
Use native String instead of System.String
Use native String instead of System.String
Use native String instead of System.String
removing unnecessary/duplicate readme file
Removing excessive, unnecessary entries, as only one entry is needed as an example.
Moving Initialise-SteamAppLookup.ps1 to Create-SteamAppLookup.ps1 to avoid variations in British English vs. American English.
Moving Initialise-SteamAppLookup.ps1 to Create-SteamAppLookup.ps1 to avoid variations in British English vs. American English.
Massive fixes and formatting changes
Massive fixes and formatting changes
Minor formatting changes
All data from ConvertFrom-VDF is now returned without double quote marks. The rest of the project has been updated since it no longer needs to handle that.
Updates for release 1.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant