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.

Sign up for GitHub

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

Jump to bottom

Remove drop util #436

Merged

RobbieTheWagner merged 1 commit into master from remove-drop

Jul 15, 2019

Member

RobbieTheWagner commented Jul 15, 2019

I think this is redundant, as we can call slice directly.


          Remove drop util

0e09c6b

I think this is redundant, as we can call `slice` directly.

RobbieTheWagner requested a review from chuckcarpenter

July 15, 2019 02:25

RobbieTheWagner added the enhancement label

chuckcarpenter approved these changes

View reviewed changes

Member

chuckcarpenter left a comment

This makes sense since drop was only used one place with no real value add to slice.

RobbieTheWagner merged commit 9dccd4e into master

RobbieTheWagner deleted the remove-drop branch

July 15, 2019 13:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels