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
chore/ci: move profiling to Actions #4535
Conversation
f3f0264
to
a2d8bfa
Compare
@@ -16,7 +16,7 @@ jobs: | |||
strategy: | |||
matrix: | |||
os: [ubuntu-latest] | |||
node-version: ['10.x', '12.x', '14.x'] | |||
node-version: ['10', '12', '14'] |
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.
Surge.sh's subdomain cannot have a dot, 10
& 10.x
seem to produce similar result.
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.
What about making benchmark & profiling as two dedicated jobs?
# benchmark.yml
jobs:
benchmark:
runs-on: ${{ matrix.os }}
#
profiling:
runs-on: ${{ matrix.os }}
#
This workflow does not work for the fork repository because it does not have permission to access the surge token. |
What does it do?
Continue #4525
Finally replace Travis.
Pull request tasks
- [ ] Add test cases for the changes.