-
Notifications
You must be signed in to change notification settings - Fork 289
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
import blocks before test #3364
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3364 +/- ##
==========================================
+ Coverage 31.74% 32.06% +0.32%
==========================================
Files 500 500
Lines 46815 46815
Branches 21312 21312
==========================================
+ Hits 14858 15005 +147
+ Misses 17809 17679 -130
+ Partials 14148 14131 -17
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Benchmark for d42a4bbClick to view benchmark
|
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.
需求是 #923
.github/workflows/build_test.yml
Outdated
- name: import blocks | ||
run: | | ||
ls -ls `pwd`/target | ||
bash ./scripts/import_block.sh main ~/.starcoin/main |
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.
这里 import_block.sh 没有用。
env: | ||
RUST_LOG: info | ||
RUST_BACKTRACE: full | ||
STARCOIN_WS: ${{ secrets.STARCOIN_WS }} |
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.
这里要用 ci 跑起来的那个节点去测试。
args: --all | ||
- name: import blocks | ||
run: | | ||
bash ./scripts/import_net_block.sh main ~/.starcoin/main |
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.
感觉应该拉个 release 版本的去测试,import 执行会很慢。
run: | | ||
bash ./scripts/import_net_block.sh main ~/.starcoin/main | ||
- name: test | ||
run: bash ./scripts/auto_rerun_test.sh |
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.
这个测试就没必要跑了。
- name: move-prover-test | ||
uses: actions-rs/cargo@v1 | ||
with: | ||
command: test | ||
args: -p starcoin-move-prover | ||
- name: check changed files | ||
run: bash ./scripts/changed_files.sh |
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.
这个测试也一样
Pull request type
Please check the type of change your PR introduces:
What is the new behavior?
import full blocks before test