Skip to content

chore: bump psake to 5.0.4#65

Merged
tablackburn merged 2 commits into
mainfrom
chore/bump-psake-5.0.4
May 29, 2026
Merged

chore: bump psake to 5.0.4#65
tablackburn merged 2 commits into
mainfrom
chore/bump-psake-5.0.4

Conversation

@tablackburn
Copy link
Copy Markdown
Owner

@tablackburn tablackburn commented May 29, 2026

Summary

  • Bump the pinned psake build dependency in build.depend.psd1 from 4.9.1 to 5.0.4 (latest stable, a major version bump).
  • Standalone validation pass — this repo first; the other module repos follow once CI here is confirmed green.

Compatibility with psake 5.0 breaking changes

The repo uses none of the removed/changed surfaces:

  • build.ps1 invokes psake with an explicit -BuildFile (the removed default-file fallback doesn't apply) and reads $psake.build_success (still supported).
  • Uses Import-Module psake; Invoke-psake (the removed psake.ps1/psake.cmd runners aren't used).
  • No OutputHandler / ColoredOutput / $framework usage.
  • The Task -FromModule … -Depends syntax still binds on 5.0.4.

Test Plan

  • Full CI green on all platforms (psake drives every job — this is the real validation)

Breaking Changes

None expected for consumers; dev/build dependency only.

Summary by CodeRabbit

  • Chores
    • Updated Pester testing dependency from version 4.9.1 to 5.0.4.

Review Change Stack

Bump the pinned psake build dependency from 4.9.1 to 5.0.4 (latest
stable). The repo uses no features removed in psake 5.0: build.ps1
invokes psake with an explicit -BuildFile and reads $psake.build_success,
and there is no OutputHandler/ColoredOutput/$framework or standalone-runner
usage. Standalone validation on this repo before propagating to the
other modules.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 29, 2026 07:35
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6ed2a447-40d5-4c98-9190-41f504bdee93

📥 Commits

Reviewing files that changed from the base of the PR and between d162f70 and 8105b02.

📒 Files selected for processing (1)
  • build.depend.psd1

📝 Walkthrough

Walkthrough

The pull request updates the Pester test framework dependency version in the build manifest from 4.9.1 to 5.0.4. This is a straightforward version bump affecting only the declared dependency constraint; no other build dependencies or configuration are modified.

Changes

Dependency Version Update

Layer / File(s) Summary
Pester version bump
build.depend.psd1
Pester dependency is updated from version 4.9.1 to 5.0.4.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop and a skip, version five takes the lead,
Pester bounds forward to test what we need,
From four-point-nine's stable to five's shiny new light,
The build grows more modern with each testing bite! 🧪✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title claims to bump 'psake' to 5.0.4, but the changeset actually updates 'Pester' from 4.9.1 to 5.0.4 in build.depend.psd1. Correct the title to accurately reflect the actual dependency being updated, such as 'chore: bump Pester to 5.0.4' instead of psake.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-psake-5.0.4

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the pinned psake build dependency used by the repository’s PowerShell build pipeline.

Changes:

  • Bumps psake from 4.9.1 to 5.0.4 in build.depend.psd1.
  • Keeps the existing PSDepend structure and build dependency configuration unchanged.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tablackburn tablackburn merged commit bb93bca into main May 29, 2026
18 of 19 checks passed
@tablackburn tablackburn deleted the chore/bump-psake-5.0.4 branch May 29, 2026 17:28
tablackburn added a commit that referenced this pull request May 30, 2026
…l reference (#67)

PSScriptAnalyzer 1.25.0 intermittently throws "Object reference not set to
an instance of an object." from inside the analyzer engine, failing the
Lint task on otherwise-green builds. It has struck both Linux (#65) and
Windows (#66) CI runners at random, so it is not platform-specific and a
non-Windows disable would not cover it.

Retry the analysis up to three times on that specific null reference, while
letting any genuine analysis error surface immediately.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

2 participants