-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Enable manual workflow trigger #1036
Enable manual workflow trigger #1036
Conversation
update codebase
update repo
update repo
update repository
add workflow_dispatch chroma-test.yml
added workflow dispatch to chroma-integration-test.yml
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
@tazarov can you take a look at this? |
How does workflow dispatch work for unauthorized users |
Here's the 1 min read - https://docs.github.com/en/actions/using-workflows/manually-running-a-workflow#configuring-a-workflow-to-run-manually The gist:
Ref: https://github.com/orgs/community/discussions/26053#discussioncomment-3250202 |
I think this can be useful. On a side note: I am unsure whether this can be triggered on a PR branch - useful for some preliminary tests before doing the full 360. |
@tazarov The idea behind this is to run unit and integration tests on a forked repository manually. This could help anyone pushing changes to manually check if changes are passing checks before making PRs or testing existing PR branches without needing maintainers to approve them after every fix. |
@sunilkumardash9, thanks for the clarification. I guess provided one enables GH Actions on a forked repo that should be possible, yes :) |
Looks great! Thanks! |
You'll have to rebase on #1054 |
update repo
Signed-off-by: sunilkumardash9 <[email protected]>
Head branch was pushed to by a user without write access
490fef4
to
afa2823
Compare
@sunilkumardash9, your rebase did not work. Can you do it again? |
updating repository
@tazarov It should be passing now. |
@sunilkumardash9 thanks for the add |
## Description of changes *Summarize the changes made by this PR.* - Added a workflow_dispatch to manually trigger test workflows - will be good for development experience --------- Signed-off-by: sunilkumardash9 <[email protected]>
## Description of changes *Summarize the changes made by this PR.* - Added a workflow_dispatch to manually trigger test workflows - will be good for development experience --------- Signed-off-by: sunilkumardash9 <[email protected]>
Description of changes
Summarize the changes made by this PR.