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

Update CI workflow to assert expected versions. #19

Merged
merged 1 commit into from
Mar 11, 2020

Conversation

rwjblue
Copy link
Collaborator

@rwjblue rwjblue commented Mar 11, 2020

Prior to this, reviewing the action output manually was required. This adds a test script that will helpfully fail the CI run if any of the expected values do not match.

@rwjblue rwjblue force-pushed the make-actions-tests-stricter branch 9 times, most recently from b7d6789 to ccea58a Compare March 11, 2020 23:30
@rwjblue
Copy link
Collaborator Author

rwjblue commented Mar 11, 2020

Rebased on top of the changes from #20 (updating the tests to be stricter caught the regression).

Prior to this, reviewing the action output manually was required. This
adds a test script that will helpfully fail the CI run if any of the
expected values do not match.
@rwjblue rwjblue force-pushed the make-actions-tests-stricter branch from 31ce364 to eacf944 Compare March 11, 2020 23:45
@rwjblue rwjblue merged commit 54845bc into master Mar 11, 2020
@rwjblue rwjblue deleted the make-actions-tests-stricter branch March 11, 2020 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant