-
Notifications
You must be signed in to change notification settings - Fork 96
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
chore(): initial 'versions.yml' #151
Conversation
b66d9d3
to
7ce000e
Compare
.github/workflows/versions.yml
Outdated
runs-on: ubuntu-latest | ||
steps: | ||
- name: Verify Setup For Tags | ||
run: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of having an individual entry for each version, I would suggest using a job matrix
.github/workflows/versions.yml
Outdated
run: | | ||
echo "Verify 0.0.2:" | ||
curl https://raw.githubusercontent.com/bpkg/bpkg/0.0.2/setup.sh | bash | ||
bpkg --version | grep 0.0.2 >/dev/null || { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something is nagging at me that this line has issues but I can't seem to get the problem in clear focus.
It might come to me later...
@@ -0,0 +1,32 @@ | |||
#!/usr/bin/env bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If a GitHub Actions matrix is used, this script will no longer be needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you know how we can get all tags into the matrix?
going to get this into a different PR |
No description provided.