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

指値注文時に設定値分不利な注文を行う機能を追加 #123

Merged
merged 2 commits into from
Mar 13, 2018
Merged

指値注文時に設定値分不利な注文を行う機能を追加 #123

merged 2 commits into from
Mar 13, 2018

Conversation

Connie-Wild
Copy link
Contributor

@Connie-Wild Connie-Wild commented Mar 12, 2018

成行注文のような動作を作れないかと思い、acceptablePriceRangeの%分だけ、不利な注文を実施する機能を追加してみました。
acceptablePriceRangeは0.5とか0.1といった低い数値を設定する事を想定しています。
もしよろしければマージをお願いします。
Testデータの作り方がいまいちわからず、申し訳ないです。

@coveralls
Copy link

coveralls commented Mar 12, 2018

Coverage Status

Coverage decreased (-0.1%) to 97.838% when pulling 9b9b2f6 on Connie-Wild:MarketOrder into 5a5c165 on bitrinjani:master.

@bitrinjani
Copy link
Owner

ありがとうございます。ロジックはよいと思います。
動作に変更はありませんが、スタイルの統一のために以下の点を修正お願いします。

@Connie-Wild
Copy link
Contributor Author

指摘ありがとうございます!
二重と三重にそんな違いがあったとは知りませんでした。勉強になりました。
あとで修正しますm(_ _)m

@bitrinjani bitrinjani merged commit 6323d70 into bitrinjani:master Mar 13, 2018
@bitrinjani
Copy link
Owner

ありがとうございました。マージしました。

@Connie-Wild
Copy link
Contributor Author

マージありがとうございます!

@Connie-Wild Connie-Wild deleted the MarketOrder branch March 13, 2018 09:10
@Connie-Wild
Copy link
Contributor Author

追加した機能の説明がREADMEに無いとイシューを立てる人が出てきそうなので、あとで説明追加分のプルリクエストをお送りします。

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