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.

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

(IAC-1307) Add Spec tests to nightly and pr_test GHActions configs #372

Merged
merged 1 commit into from
Mar 10, 2021
Merged

(IAC-1307) Add Spec tests to nightly and pr_test GHActions configs #372

merged 1 commit into from
Mar 10, 2021

Conversation

sanfrancrisko
Copy link
Contributor

@sanfrancrisko sanfrancrisko commented Dec 11, 2020

Description

This change adds a new workflow to run spec (unit) tests as a
standalone manual job and as options to the 'Nightly' and 'PR'
tests.

Testing

Pre-requisites before merging

@sanfrancrisko sanfrancrisko requested review from a team as code owners December 11, 2020 11:38
@sanfrancrisko sanfrancrisko marked this pull request as draft December 11, 2020 11:38
@sanfrancrisko
Copy link
Contributor Author

sanfrancrisko commented Dec 11, 2020

Thinking this could do with some more intelligence and dynamic generation of job steps so we don't have to do another rollout when Puppet 5 is deprecated, making use of this change: puppetlabs/puppet_litmus#357

@DavidS
Copy link
Contributor

DavidS commented Jan 18, 2021

Rebased to see if it all works now

@CLAassistant
Copy link

CLAassistant commented Feb 24, 2021

CLA assistant check
All committers have signed the CLA.

This change adds a new workflow to run spec (unit) tests as a
standalone manual job and as options to the 'Nightly' and 'PR'
tests.
Copy link
Contributor

@Disha-maker Disha-maker left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Disha-maker Disha-maker merged commit 486d0d3 into puppetlabs:main Mar 10, 2021
@sanfrancrisko sanfrancrisko deleted the IAC-1307/main/spec_tests_ghactions branch March 10, 2021 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants