feat: auto-trigger code sample testing#1367
feat: auto-trigger code sample testing#1367m-murasovs wants to merge 17 commits intoapify:masterfrom
Conversation
There was a problem hiding this comment.
This sample is added so that the workflow gets triggered - it's also added in another PR
|
|
||
| - name: Call Code Sample Tester Actor | ||
| env: | ||
| APIFY_TOKEN: ${{ secrets.CODE_SAMPLE_TESTER_ACTOR_TOKEN }} |
There was a problem hiding this comment.
We'll need to add a token for this once the actor is moved to the Apify account
| curl --fail-with-body -X POST "https://api.apify.com/v2/acts/X8D0L4wfpO8fCL1uL/run-sync" \ | ||
| -H "Content-Type: application/json" \ | ||
| -H "Authorization: Bearer $APIFY_TOKEN" \ | ||
| -d '{ "branch": "'"$BRANCH_NAME"'" }' |
There was a problem hiding this comment.
Will this work on PRs from forks?
There was a problem hiding this comment.
The run in the console suggests otherwise
There was a problem hiding this comment.
Great catch! I've modified the action and the actor so that the owner of the triggering repo is passed
There was a problem hiding this comment.
Here's the last run with the test: https://console.apify.com/actors/X8D0L4wfpO8fCL1uL/runs/g4B6nhuQZVKlOc9dx#log
There was a problem hiding this comment.
Also, sorry I deleted that comment because I realised it was wrong, but hadn't seen your second one yet :)
2 participants
No description provided.