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

Fix issues with GHA config #449

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

Fix issues with GHA config #449

wants to merge 21 commits into from

Conversation

vexx32
Copy link
Owner

@vexx32 vexx32 commented May 19, 2021

PR Summary

Fixing some issues in the GHA config.

Context

Changes

Checklist

  • Pull Request has a meaningful title.
  • Summarised changes.
  • Pull Request is ready to merge & is not WIP.
  • Added tests / only testable interactively.
    • Make sure you add a new test if old tests do not effectively test the code changed.
  • Added documentation / opened issue to track adding documentation at a later date.

@vexx32 vexx32 requested a review from steviecoaster as a code owner May 19, 2021 14:19
@vexx32 vexx32 added the Category-Build 🚀 Modifies the build / deployment system label May 19, 2021
vexx32 added 20 commits May 19, 2021 10:24
@vexx32
🚀 Fix syntax.. I think
e70777b
@vexx32
🚀 Update syntax to fix things
1683f08
@vexx32
🚀 Adjust expression syntax
c5b0266
@vexx32
🚀 add publish step name
634a148
@vexx32
🚀 Update Build-Module to support GHA
b07f621
@vexx32
🚀 update env var casing
bc776b3
@vexx32
🚀 Adjust initialize-environment.ps1 for gha
99e92b0
@vexx32
🚀 update env init
a14a5d1
@vexx32
🚀 fix env var ref in build
a286827
@vexx32
🚀 fix env var usage
9b9c559 
casing issue suuuuuck
@vexx32
🚀 sigh
623220b
@vexx32
🚀 fix variable refs
6ee6cbd 
grumbling about misleading documentation
@vexx32
🚀 fix publish with multiple pester versions
aaccca1 
when more than one pester version is available, publishing fails because powershellget is awful
@vexx32
🚀 fix artifact handling for Pester version
b09e470
@vexx32
🚀 Force CI to use the declared version dep
87404b8
@vexx32
🚀 force ci tests to use exact pester ver
af5db2a
@vexx32
🚀 Force pskoans to use pester 5.0.2
9a041c6 
testing a theeery
@vexx32
🚀 fix version detection
1d73ab2
@vexx32
🚀 fix dep version
ff1a7f1
@vexx32
🚀 fix pester version check
c2b9db1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@steviecoaster steviecoaster Awaiting requested review from steviecoaster steviecoaster is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Category-Build 🚀 Modifies the build / deployment system
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@vexx32