-
Notifications
You must be signed in to change notification settings - Fork 6
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
Document example workflows with upload and also test them #136
Conversation
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.
Thanks for the PR. I left a couple of comments. Apart from those, should we consider offering the two usage cases as separate workflow files? something like usage-current-repository.yml
and usage-multiple-repositories.yml
? (we can leave that for another day, just wanted to get your opinion)
…g with unzipping/webbrowser
Thanks for the comments, I addressed all of them. |
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.
Good stuff, thank you Stefan!
Refs #79
Description
.github/workflows/usage.yml
..github/workflows/usage.yml
workflow no longer runs on each PR as.github/workflows/testing.yml
tests the action much better. It can still be triggered manually see https://github.com/tortellini-tools/action/actions/runs/927859407 and will be run when main branch is changed.Before you get started
Checklist
This section should be filled in by the creator of this pull request to make sure the pull request is ready to review.
Instructions to review the pull request