-
Notifications
You must be signed in to change notification settings - Fork 59
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
feat(package): build es version compatible with node #1371
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1371 +/- ##
===========================================
+ Coverage 88.18% 88.26% +0.07%
===========================================
Files 56 56
Lines 3098 3101 +3
Branches 39 39
===========================================
+ Hits 2732 2737 +5
+ Misses 345 343 -2
Partials 21 21
Continue to review full report at Codecov.
|
.github/workflows/main.yml
Outdated
fetch-depth: 100 | ||
- if: github.ref != 'refs/heads/develop' | ||
run: git branch develop origin/develop | ||
fetch-depth: 50 |
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.
why do we need fetch-depth at all?
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.
ah I see, to lint the last 50 commits
src/chain/index.js
Outdated
deepProps: { Ae: { defaults: { waitMined: true, verify: true } } } | ||
deepProps: { | ||
Ae: { | ||
defaults: { |
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.
why is this needed here?
src/chain/index.js
Outdated
} | ||
}, | ||
methods: { | ||
_getPollInterval (type) { |
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.
why is this needed here?
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.
so, these all are changes from #1369, I've rebased this PR on the top of develop and they are not visible in the diff anymore
21d475a
to
480c747
Compare
depends on #1369 (needed for cli)
So, after this would be possible to do:
and execute it using plain node: