-
Notifications
You must be signed in to change notification settings - Fork 0
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: PSModule/Test-PSModule
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 331d42c
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: PSModule/Test-PSModule
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 1fd648f
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 4 commits
- 10 files changed
- 1 contributor
Commits on Jan 11, 2025
-
🩹 [Patch]: Add Retry on Install-Module (#88)
## Description This pull request introduces changes to the `Resolve-PSModuleDependency.ps1` script to improve module installation reliability by adding retry logic. Enhancements to module installation: * [`scripts/helpers/Resolve-PSModuleDependency.ps1`](diffhunk://#diff-48557b471e7d62a89be3627235334ab8a39eb6119174abb61714b92d844508e2L1-R3): Added a `#Requires -Modules Retry` directive to ensure the `Retry` module is available. * [`scripts/helpers/Resolve-PSModuleDependency.ps1`](diffhunk://#diff-48557b471e7d62a89be3627235334ab8a39eb6119174abb61714b92d844508e2R52-R54): Wrapped the `Install-Module` command with retry logic, attempting the installation up to 5 times with a 10-second delay between attempts. ## Type of change <!-- Use the check-boxes [x] on the options that are relevant. --> - [ ] 📖 [Docs] - [ ] 🪲 [Fix] - [x] 🩹 [Patch] - [ ]
⚠️ [Security fix] - [ ] 🚀 [Feature] - [ ] 🌟 [Breaking change] ## Checklist <!-- Use the check-boxes [x] on the options that are relevant. --> - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areasConfiguration menu - View commit details
-
Copy full SHA for 7aaef7b - Browse repository at this point
Copy the full SHA 7aaef7bView commit details
Commits on Jan 30, 2025
-
🪲 [Fix]: Fix script path and improve test result handling (#91)
## Description This pull request includes changes to the GitHub Actions workflow and the PowerShell script used for testing. The most important changes include updating the script path in the action configuration and modifying the handling of test results in the PowerShell script. ### Workflow Updates: * [`action.yml`](diffhunk://#diff-1243c5424efaaa19bd8e813c5e6f6da46316e63761421b3e5f5c8ced9a36e6b6L72-R72): Updated the script path in the `runs:` section to use a more straightforward syntax. ### Script Enhancements: * [`scripts/main.ps1`](diffhunk://#diff-dc2e5a659836b1b73abb03421c567f5018c2755677c4a0aa764cb26117b68011R53-R63): Improved the handling of test results by setting GitHub output values for passed tests and using a return variable to determine the exit code. ## Type of change <!-- Use the check-boxes [x] on the options that are relevant. --> - [ ] 📖 [Docs] - [x] 🪲 [Fix] - [ ] 🩹 [Patch] - [ ]
⚠️ [Security fix] - [ ] 🚀 [Feature] - [ ] 🌟 [Breaking change] ## Checklist <!-- Use the check-boxes [x] on the options that are relevant. --> - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areasConfiguration menu - View commit details
-
Copy full SHA for e96fb15 - Browse repository at this point
Copy the full SHA e96fb15View commit details -
🪲 [Fix]: Fix conditional logic for test result handling (#92)
## Description This pull request includes a small change to the `scripts/main.ps1` file. The change modifies the conditional logic to use an `elseif` statement instead of a separate `if` statement, ensuring that the script correctly handles the cases where tests have failed or all tests have passed. * [`scripts/main.ps1`](diffhunk://#diff-dc2e5a659836b1b73abb03421c567f5018c2755677c4a0aa764cb26117b68011L55-R55): Changed the conditional logic from a separate `if` statement to an `elseif` statement to handle test results more accurately. ## Type of change <!-- Use the check-boxes [x] on the options that are relevant. --> - [ ] 📖 [Docs] - [x] 🪲 [Fix] - [ ] 🩹 [Patch] - [ ]
⚠️ [Security fix] - [ ] 🚀 [Feature] - [ ] 🌟 [Breaking change] ## Checklist <!-- Use the check-boxes [x] on the options that are relevant. --> - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areasConfiguration menu - View commit details
-
Copy full SHA for 49dd4d7 - Browse repository at this point
Copy the full SHA 49dd4d7View commit details
Commits on Feb 16, 2025
-
🩹 [Patch]: Update linter and git configuration (#97)
## Description This pull request includes several updates to configuration files and scripts, primarily focusing on code quality and compliance improvements. The most significant changes include updating linter configurations, modifying PowerShell script settings, and updating the license year. ### Linter Configuration Updates: * Added a new `.jscpd.json` configuration file to set up code duplication detection with specific thresholds and ignore patterns. * Updated `.powershell-psscriptanalyzer.psd1` to enable specific rules and adjust settings for PowerShell script analysis. * Updated `.github/workflows/Linter.yml` to disable JSON Prettier validation. ### PowerShell Script Updates: * Added `SuppressMessageAttribute` to suppress specific PSScriptAnalyzer rules in `Resolve-PSModuleDependency.ps1` and `Test-PSModule.ps1`. [[1]](diffhunk://#diff-48557b471e7d62a89be3627235334ab8a39eb6119174abb61714b92d844508e2R21-R25) [[2]](diffhunk://#diff-2100eb52126417766eeb5b98f9c6a0da1dd10302b3b7c05d476295e3bd76ce76R12-R15) * Added `SuppressMessageAttribute` to the main script `main.ps1` to suppress the `PSAvoidUsingWriteHost` rule. ### License Update: * Updated the license year in the `LICENSE` file from 2024 to 2025. ## Type of change <!-- Use the check-boxes [x] on the options that are relevant. --> - [ ] 📖 [Docs] - [ ] 🪲 [Fix] - [x] 🩹 [Patch] - [ ]
⚠️ [Security fix] - [ ] 🚀 [Feature] - [ ] 🌟 [Breaking change] ## Checklist <!-- Use the check-boxes [x] on the options that are relevant. --> - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areasConfiguration menu - View commit details
-
Copy full SHA for 1fd648f - Browse repository at this point
Copy the full SHA 1fd648fView commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 331d42c...1fd648f