Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix the unused variable warnings in tests #851

Merged
merged 2 commits into from
Apr 17, 2018
Merged

Fix the unused variable warnings in tests #851

merged 2 commits into from
Apr 17, 2018

Conversation

ihla
Copy link

@ihla ihla commented Apr 16, 2018

No description provided.

@oxarbitrage
Copy link
Member

market_tests and operation_tests should wait as we are working on this files for hardfork and can create conflicts.

if you remove this 2 i have no issue on the other 2 test changes and merge now.

thanks.

@oxarbitrage oxarbitrage merged commit 49aeb22 into bitshares:develop Apr 17, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@oxarbitrage oxarbitrage oxarbitrage approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
201806 - Non-Consensus-Changing Release
Development

Successfully merging this pull request may close these issues.
3 participants
@ihla @oxarbitrage @abitmore