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
Fixing some imports issues with bwcTsMigration #2338
Conversation
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.
ACK! Nice work!
@@ -4,8 +4,8 @@ | |||
"author": "BitPay Inc", | |||
"version": "8.6.0", | |||
"license": "MIT", | |||
"main": "ts_build/index.js", | |||
"types": "./ts_build/index.d.ts", | |||
"main": "./ts_build/index.js", |
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.
@justinkook remove these?
}, | ||
"scripts": { | ||
"start": "npm run clean && npm run tsc && node app.js", | ||
"coverage": "./node_modules/.bin/istanbul cover ./node_modules/.bin/_mocha -- --reporter spec test", | ||
"test": "./node_modules/.bin/mocha --exit", | ||
"test": "npm run compile && ./node_modules/.bin/mocha --exit", | ||
"test:ci": "npm run test", | ||
"docs": "./node_modules/.bin/jsdox lib/* lib/common lib/errors -o docs && cat README.header.md docs/*.md LICENSE > README.md", | ||
"coveralls": "./node_modules/.bin/istanbul cover ./node_modules/mocha/bin/_mocha --report lcovonly -- -R spec && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js && rm -rf ./coverage", |
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.
export const Common = { Constants, Defaults, Utils }; | ||
/* | ||
*module.exports = { |
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.
Commented out code
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.
ACK
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.
Tested and works on Copay. One interesting thing is that in package.json
if "main": "./ts_build/index.js",
the ts_build folder is missing after npm i
.
It works if "main": "ts_build/index.js",
Couldn't figure out how to submit this pr to your fork @Gamboster