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

minor test code cleanup #17645

Merged
merged 1 commit into from
Jun 2, 2021
Merged

Conversation

jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented Jun 1, 2021

Problem

Superfluous type restriction leads to inflexibility when the types change (which they will).

Summary of Changes

get rid of unnecessary type.
Fixes #

@codecov
Copy link

codecov bot commented Jun 1, 2021

Codecov Report

Merging #17645 (efb271c) into master (886898e) will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master   #17645   +/-   ##
=======================================
  Coverage    82.7%    82.7%           
=======================================
  Files         426      426           
  Lines      119416   119421    +5     
=======================================
+ Hits        98814    98823    +9     
+ Misses      20602    20598    -4     

@jeffwashington jeffwashington marked this pull request as ready for review June 2, 2021 00:34
Copy link
Contributor

@lijunwangs lijunwangs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jeffwashington jeffwashington merged commit 107af52 into solana-labs:master Jun 2, 2021
mergify bot pushed a commit that referenced this pull request Jun 2, 2021
(cherry picked from commit 107af52)
mergify bot added a commit that referenced this pull request Jun 2, 2021
(cherry picked from commit 107af52)

Co-authored-by: Jeff Washington (jwash) <[email protected]>
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
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.

2 participants