Skip to content

Conversation

@ReeceGoding
Copy link
Contributor

Type of Change

  • Bug fix (non-breaking change, fixes Invoke-ManualPester claims that it supports -Show All #10144 )
  • New feature (non-breaking change, adds functionality, fixes # )
  • Breaking change (affects multiple commands or functionality, fixes # )
  • Ran manual Pester test and has passed (Invoke-ManualPester)
  • Adding code coverage to existing functionality
  • Pester test is included
  • If new file reference added for test, has is been added to github.com/dataplat/appveyor-lab ?
  • Unit test is included
  • Documentation
  • Build system

Purpose

Made docs for Invoke-ManualPester match the ValidateSet on Show. Closes #10144. Documentation only change.

Copy link
Contributor

@niphlod niphlod left a comment

Choose a reason for hiding this comment

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

I guess this is good. @potatoqualitee , @andreasjordan , do we have still tests with pester4 or are they all been ported succesfully to pester5 ?
If so, we can go ahead and remove code from here that can run pester4 tests

@andreasjordan
Copy link
Collaborator

We only use pester 5

@niphlod
Copy link
Contributor

niphlod commented Feb 9, 2026

Perfect then, we can cut out lot of the cruft that made sure both "versions" could be run

@potatoqualitee potatoqualitee merged commit 1cdc5d2 into dataplat:development Feb 11, 2026
4 checks passed
@potatoqualitee
Copy link
Member

Thanks all!

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.

Invoke-ManualPester claims that it supports -Show All

4 participants