push!(df, row) for adding rows to a DataFrame

[catawbasam]

No description provided.

[johnmyleswhite]

Let's do this as Base.push! rather than importing push!

[catawbasam]

ok

[johnmyleswhite]

Looks good to me. If you've got time, that one change is all I'd suggest doing.

[catawbasam]

Rats. Not sure how these Travis details relate to the pull request. Any thoughts?

$ julia --code-coverage ./test/runtests.jl
ERROR: DataFrames not found
 in require at loading.jl:47
 in include at boot.jl:244
 in include_from_node1 at loading.jl:128
while loading /home/travis/build/JuliaStats/DataFrames.jl/test/runtests.jl, in expression starting on line 9

[johnmyleswhite]

I don't see how they could be related. The tests run fine on your local machine?

[catawbasam]

Yup:

[keithc@cockpit1 DataFrames]$ julia --code-coverage ./test/runtests.jl
Running tests:
    PASSED: utils.jl
    PASSED: data.jl
    PASSED: index.jl
    PASSED: dataframe.jl
    PASSED: io.jl
    PASSED: constructors.jl
    PASSED: RDA.jl
    PASSED: sort.jl
    PASSED: grouping.jl
    PASSED: join.jl
    PASSED: vcat.jl
    PASSED: iteration.jl
    PASSED: duplicates.jl
    PASSED: show.jl

[johnmyleswhite]

Let's ignore it for now and merge away.