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

test (perp): Add test for remove margin #595

Merged
merged 7 commits into from
Jun 13, 2022

Conversation

kai0x
Copy link
Contributor

@kai0x kai0x commented Jun 12, 2022

i remade this pr to address the merge conflicts

this addresses all the previous comments

x/perp/client/cli/cli_test.go Outdated Show resolved Hide resolved
@AgentSmithMatrix AgentSmithMatrix self-requested a review June 13, 2022 07:55
@AgentSmithMatrix AgentSmithMatrix merged commit 73ceb1f into master Jun 13, 2022
@AgentSmithMatrix AgentSmithMatrix deleted the 520-remove-margin-new-master branch June 13, 2022 07:56
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.

cli: RemoveMargin on an underwater position should error out and realize losses
3 participants