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

.github: workflows: add workflow to test fetch-configlet scripts #690

Merged
merged 6 commits into from
Oct 31, 2022

Conversation

ee7
Copy link
Member

@ee7 ee7 commented Oct 26, 2022

The fetch-configlet scripts were previously not tested during CI. Test them.

The fetch-configlet.ps1 script currently always fetches the Windows release asset:

$fileName = "configlet-windows-$arch.zip"

so this workflow only tests that script on Windows. (Issue #361 tracks this low-priority problem).

Closes: #121

ee7 added 3 commits October 26, 2022 20:30
The `fetch-configlet.ps1` script currently always fetches the Windows
release asset, so this workflow only tests that script on Windows.

Closes: 121
@ee7
Copy link
Member Author

ee7 commented Oct 26, 2022

@ee7 ee7 requested a review from ErikSchierboom October 28, 2022 12:30
@ee7 ee7 merged commit 0be1889 into exercism:main Oct 31, 2022
@ee7 ee7 deleted the github-workflows-fetch-configlet branch October 31, 2022 21:41
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.

fetch-configlet: add test
2 participants