Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Sample random trade_infos for success #4043

Merged
merged 1 commit into from
Apr 28, 2019

Conversation

sakridge
Copy link
Contributor

Problem

Just looking at a single trade_info which may or may not succeed can fail often.

Summary of Changes

Sample multiple trade_infos for success.

Fixes #

Just looking at a single trade_info which may or may not succeed
can fail often.
@sakridge sakridge requested a review from jackcmay April 27, 2019 23:08
@codecov
Copy link

codecov bot commented Apr 27, 2019

Codecov Report

Merging #4043 into master will decrease coverage by <.1%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master   #4043     +/-   ##
========================================
- Coverage    77.7%   77.7%   -0.1%     
========================================
  Files         164     164             
  Lines       27892   27894      +2     
========================================
- Hits        21697   21694      -3     
- Misses       6195    6200      +5

1 similar comment
@codecov
Copy link

codecov bot commented Apr 27, 2019

Codecov Report

Merging #4043 into master will decrease coverage by <.1%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master   #4043     +/-   ##
========================================
- Coverage    77.7%   77.7%   -0.1%     
========================================
  Files         164     164             
  Lines       27892   27894      +2     
========================================
- Hits        21697   21694      -3     
- Misses       6195    6200      +5

@sakridge sakridge merged commit 4e5677f into solana-labs:master Apr 28, 2019
@sakridge sakridge deleted the exchange-trade-infos branch April 28, 2019 18:00
steveluscher pushed a commit to alessandrod/solana that referenced this pull request Dec 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant