-
Notifications
You must be signed in to change notification settings - Fork 17
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 e2e yml for github actions #195
Add e2e yml for github actions #195
Conversation
.github/workflows/e2e.yml
Outdated
|
||
steps: | ||
# Pinned 1.0.0 version | ||
- uses: haya14busa/action-workflow_run-status@967ed83efa565c257675ed70cfe5231f062ddd94 |
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.
use marocchino/action-workflow_run-status@54b6e87d6cb552fc5f36dbe9a722a6048725917a
to avoid node 12 warnings which is deprecated.
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.
Please update this workflow as well:
- uses: haya14busa/action-workflow_run-status@967ed83efa565c257675ed70cfe5231f062ddd94 |
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.
change done, as suggested.
.github/workflows/build.yml
Outdated
@@ -29,7 +29,7 @@ jobs: | |||
|
|||
steps: | |||
# Pinned 1.0.0 version | |||
- uses: haya14busa/action-workflow_run-status@967ed83efa565c257675ed70cfe5231f062ddd94 | |||
- uses: marocchino/action-workflow_run-status@54b6e87d6cb552fc5f36dbe9a722a6048725917a | |||
- uses: actions/checkout@v3 | |||
with: | |||
ref: ${{ github.event.workflow_run.head_branch }} |
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.
Although unrelated to this PR, this should be sha
instead of branch
. Please fix.
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.
Updated the change
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.
Please squash commits before merge.
6727491
to
0c2aa4c
Compare
Squash commits done. |
Add e2e yml for github actions