JuliaData / DataFrames.jl / 5758 / 6
93%
master: 93%

DEFAULT BRANCH: master
Ran
Files 23
Run time 5s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 94.574% (+0.2%) from 94.414%
5758.6

push

travis-ci

web-flow 
[BREAKING] Add column indexing using strings

User visible changes:
* allow using an AbstractString everywhere where Symbol was accepted as a column indicator (i.e. this extends beyond just getindex and getproperty, every function was reviewed and made consistent)
* docstrings were updated to reflect this change + some unrelated docs fixes were added where spotted
* names function consistently returns vector of strings for all types that return column names
* propertynames function consistently returns vector of Symbols for all types that return column names
* keys function consistently returns vector of Symbols for all types that return column names as keys
* depend on DataAPI.jl v1.2
* rename! passes String to a function if API with function is used
* consistently define hasproperty for types that have custom getproperty
* add cols to names function for types where it makes sense consistently
* push! accepts dicts with string keys
* indexing with an arbitrary eltype vector now throws better error messages (informtive ArgumentErroris thrown in more cases rather thanMethodError` in the past)

Internal changes:
* add definitions of SymbolOrString, ColumnIndex (updated), and MultiColumnIndex unions and use them consistently across whole codebase
* remove keys from AbstractIndex as it was never used
* correct formatting of code in many places (mainly docstrings, function signatures, and adding missing return statements in long functions as many of them were updated anyway, but also in unrelated places)
* fix a rule that _names returns vector of Symbols without copying
* in some places switch from names to _names internally to avoid unnecessary allocations
* in some places use groupcols where this function would be an appropriate to call; similarly with names(df, cols) where it made sense
* remove without function that was not needed anymore

Co-authored-by: Milan Bouchet-Valat <nalimilan@club.fr>
Co-authored-by: pdeffebach <2319622... (continued)

3294 of 3483 relevant lines covered (94.57%)

340707.29 hits per line

Source Files on job 5758.6