Skip to content
Permalink

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
Choose a base ref
...
head repository: PSModule/Test-PSModule
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 1fd648f
Choose a head ref
  • 4 commits
  • 10 files changed
  • 1 contributor

Commits on Jan 11, 2025

  1. 🩹 [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 areas
    MariusStorhaug authored Jan 11, 2025
    Configuration menu
    Copy the full SHA
    7aaef7b View commit details
    Browse the repository at this point in the history

Commits on Jan 30, 2025

  1. 🪲 [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 areas
    MariusStorhaug authored Jan 30, 2025
    Configuration menu
    Copy the full SHA
    e96fb15 View commit details
    Browse the repository at this point in the history
  2. 🪲 [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 areas
    MariusStorhaug authored Jan 30, 2025
    Configuration menu
    Copy the full SHA
    49dd4d7 View commit details
    Browse the repository at this point in the history

Commits on Feb 16, 2025

  1. 🩹 [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 areas
    MariusStorhaug authored Feb 16, 2025
    Configuration menu
    Copy the full SHA
    1fd648f View commit details
    Browse the repository at this point in the history
Loading