-
Notifications
You must be signed in to change notification settings - Fork 35
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
Add run_doc_benchmark #534
Conversation
COMMIT_SHA=$(git rev-parse HEAD) | ||
curl -X POST ${{ secrets.SLACK_URL}} \ | ||
--data-urlencode \ | ||
"payload={'text':'[BMT Request] ${{ github.event.repository.html_url }}/commit/${COMMIT_SHA}'}" |
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.
perhaps, there is URL missing
curl: no URL specified!
curl: try 'curl --help' or 'curl --manual' for more information
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.
The url is slack webhook url, so it should be encapsulated..
So I used repository -> secrets for hiding webhook url, and this can't be triggered from PR as I know.
Please check below.
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. The workflow works.
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.
Agree
Description
This PR is for process CredData DOC BMT.
CredData DOC BMT is done internally, and just update the commit status when PR timing.
You can check the result through the commit status of b8bdb1a at csh519#8.
How has this been tested?