Create package and setup boilerplate for lisk-transaction-pool - Closes #870 #908
Create package and setup boilerplate for lisk-transaction-pool - Closes #870 #908
Conversation
@@ -0,0 +1,6 @@ | |||
{ | |||
"env": { | |||
"mocha": true, |
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.
I don't think we need browsertest
for transaction-pool.
It's a server-side only feature
@@ -0,0 +1,5 @@ | |||
{ | |||
"video": false, | |||
"pluginsFile": false, |
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.
we don't need cypress as we don't have browsertest
"url": "https://github.com/LiskHQ/lisk-elements/issues" | ||
}, | ||
"engines": { | ||
"node": ">=6.3 <=10", |
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.
node minimum is now 8.10
"test:watch": "npm test -- --watch", | ||
"test:watch:min": "npm run test:watch -- --reporter=min", | ||
"test:node": "npm run build:check", | ||
"serve:start": "http-server -p 11542 ./browsertest &", |
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.
we can delete all browser related stuff
"chai": "4.1.2", | ||
"cypress": "3.1.0", | ||
"eslint": "5.5.0", | ||
"eslint-config-airbnb-base": "13.1.0", |
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.
we can delete browser related stuff + eslint and babel
rm -rf .nyc_output/ | ||
rm -rf browsertest.build/ | ||
rm -rf coverage/ | ||
rm -rf dist-browser/ |
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.
you can remove unneeded folder delete commands here
"@types/node": "10.10.1", | ||
"bip39": "^2.5.0", | ||
"chai": "4.1.2", | ||
"cypress": "3.1.0", |
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.
you dont need cypress too
}, | ||
"dependencies": {}, | ||
"devDependencies": { | ||
"@types/bip39": "^2.4.0", |
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.
bip39
would not be needed too
}, | ||
"engines": { | ||
"node": ">=8.10 <=10", | ||
"npm": ">=3 <=6" |
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.
other packages list "npm":">=5"
Create a new package for
@lisk/lisk-transaction-pool
, add npm scripts for running application and for running testing. Furthermore, add the skeleton code for transaction pool.