Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update CI pipeline to use Pester v5 #478

Closed
BernieWhite opened this issue Jun 1, 2020 · 2 comments · Fixed by #815
Closed

Update CI pipeline to use Pester v5 #478

BernieWhite opened this issue Jun 1, 2020 · 2 comments · Fixed by #815
Assignees
@ArmaanMcleod
Labels
ci-quality Issues that affect CI/ CD processes help wanted Extra attention is needed
Milestone
v1.8.0

Comments

@BernieWhite
Copy link
Member

Pester v5 has just been released and introduced a number of breaking changes. These changes are not compatible with the current pipeline configuration.

For the moment Pester v4 will be used, however it should be upgraded to v5 at some time in the future.
@BernieWhite BernieWhite added the ci-quality Issues that affect CI/ CD processes label Jun 1, 2020
@BernieWhite BernieWhite added the help wanted Extra attention is needed label Aug 17, 2020
@ArmaanMcleod
Copy link
Contributor

@BernieWhite Can assign this to me. I can migrate the pester tests to v5 just like in PSRule.Rules.Azure 🙂

@ArmaanMcleod ArmaanMcleod mentioned this issue Sep 30, 2021
Merged
7 tasks
@BernieWhite
Copy link
Member Author

@ArmaanMcleod Thanks Armaan.

@ArmaanMcleod ArmaanMcleod mentioned this issue Oct 2, 2021
Merged
7 tasks
@BernieWhite BernieWhite added this to the v1.8.0 milestone Oct 3, 2021
This was referenced Oct 4, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ArmaanMcleod ArmaanMcleod

Labels
ci-quality Issues that affect CI/ CD processes help wanted Extra attention is needed
Projects
None yet
Milestone
v1.8.0
Development

Successfully merging a pull request may close this issue.

Pester v5 migration ArmaanMcleod/PSRule
2 participants
@BernieWhite @ArmaanMcleod