Skip to content

CLN: remove old vb_suite files (now asv_bench) #16034

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

Merged

Conversation

jorisvandenbossche
Copy link
Member

Removed the full vb_suite directory. We don't use it anymore, everything has been converted to the asv benchmarks, and we can always find it in the git history.

@jreback
Copy link
Contributor

jreback commented Apr 17, 2017

hmm thought there was an open issue / check box for this somewhere, did a quick search but didn't find it

@codecov
Copy link

codecov bot commented Apr 17, 2017

Codecov Report

Merging #16034 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #16034   +/-   ##
=======================================
  Coverage      91%      91%           
=======================================
  Files         153      153           
  Lines       50469    50469           
=======================================
  Hits        45927    45927           
  Misses       4542     4542
Flag Coverage Δ
#multiple 88.76% <ø> (ø) ⬆️
#single 40.43% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 89bd268...6628956. Read the comment docs.

@jorisvandenbossche jorisvandenbossche merged commit 1a09437 into pandas-dev:master Apr 17, 2017
@jorisvandenbossche jorisvandenbossche deleted the remove-vb_suite branch April 17, 2017 20:27
@jorisvandenbossche jorisvandenbossche added this to the 0.20.0 milestone Apr 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants