Conversation
…ed behind base64 for security reasons, also updating test cases and changing the way some of the code works to support better test cases.
…de duplication and refactored some code to be less complex by splitting them into their own files.
…for psscriptanalyzer, standardizing on write-statusmessage as well
…ile so it conforms to the format expected, more test cases and some refactoring and elimination of duplicate code.
…run functionality across the various write operations
…ds 1 comment so we will use linux
1.0.10 features expanded unit tests added dryrun functionality, around 80% done added -explain command to devsetup added environment file validation removed a decent amount of code duplication
Automated Release Tagging for 1.0.10 in DevSetup.psd1
….0.100 because it lacked multiple number parsing i.e. [0-9]+
…version-numbers Fixing an issue with parsing of version numbers where 1.0.10 became 1…
…at use the new structure Added parity code to ensure dryrun works properly with powershell modules Refactored code to be more in line with our guidelines and removed code that is no longer being used, making the functions simpler in general Updated test cases to maintain 100% test coverage
…for-parity Updated powershell provider to be in parity with the other modules that use the new structure
Refactored code to be in line with the other providers Refactored the test cases to cover this new or removed functionality Ensured DryRun compatibilityAdded a new helper function to Find-Chocolatey rather then relying on Get-Command Removed all references to Invoke-Expression and replaced with Invoke-Command
The Linux and macOS test cases had some issues with lack of TestDrive and Join-Path
Corrected issue with blank paths on the test cases for linux
…parity setting up parity with other providers adjusted multiple test cases for chocolatey refactored code chocolatey code to be more efficient with less duplication added a helper function to find the choco binary rather then relying on just Get-Command or hard coding 'choco.exe' for those times when its not in the path
feat/self updater - Allows for easy upgrade to newer versions of devsetup.
Adding self updater
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
PSScriptAnalyzer found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
Pester test (On macOS) Results1 634 tests 1 633 ✅ 54s ⏱️ Results for commit 3dc1668. |
Summary - Pester Test Coverage ReportSummary
CoverageDevSetup - 0%
DevSetup/Private/3rdParty - 100%
DevSetup/Private/3rdParty/VisualStudio - 100%
DevSetup/Private/3rdParty/VisualStudioCode - 100%
DevSetup/Private/Commands - 100%
DevSetup/Private/Enums - 100%
DevSetup/Private/Providers/Chocolatey - 100%
DevSetup/Private/Providers/Core - 100%
DevSetup/Private/Providers/Homebrew - 100%
DevSetup/Private/Providers/Powershell - 100%
DevSetup/Private/Providers/Scoop - 100%
DevSetup/Private/Updater - 100%
DevSetup/Private/Utils - 100%
DevSetup/Public - 100%
|
Pester test (On Windows) Results1 634 tests 1 633 ✅ 54s ⏱️ Results for commit 3dc1668. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merging to main to release 1.0.10
feat/self-updater
feat/explain command
feat/dry run functionality
fix/test cases to achieve close to100% coverage
fix/display issues