Skip to content
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

getBalanceがconfigのenabledを参照していない問題の修正 #3

Merged
merged 1 commit into from
Dec 14, 2017

Conversation

8823-scholar
Copy link
Contributor

目的

yarn getBalance 実行時に、enabled: false の broker まで対象にしていたので、エラーを起こしていたのを修正

また、自分のローカルで、getBalance.ts のコンパイルが通らなかったので、調整を加えさせていただきました。

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.947% when pulling b2b3bf2 on 8823-scholar:issues/get-balance into efa288e on bitrinjani:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.947% when pulling aedcaed on 8823-scholar:issues/get-balance into efa288e on bitrinjani:master.

@bitrinjani
Copy link
Owner

getBalanceの変更はokですが、package.json, tsconfig.jsonの変更は不要かと思います。
tsconfigのtargetがES6のとき、ES6の
libはデフォルトで含まれます。
また、Promiseの型情報もすでに含まれているはずです。

@8823-scholar
Copy link
Contributor Author

確認ありがとうございます!
なるほどー。
自分のvimが別のtsconfigを参照しちゃってるのかもですね。
ひとまず、このPRからは除去させていただきました 🙇

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.947% when pulling 406f4a1 on 8823-scholar:issues/get-balance into efa288e on bitrinjani:master.

@bitrinjani bitrinjani merged commit 7ee581c into bitrinjani:master Dec 14, 2017
@bitrinjani
Copy link
Owner

提案・修正ありがとうございます。マージしました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants