Skip to content

Commit

Permalink
Remove FIXME about test/indexing.jl
Browse files Browse the repository at this point in the history
Better handle that separately.
  • Loading branch information
nalimilan committed Aug 30, 2016
1 parent 1da6116 commit beab3a9
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
6 changes: 0 additions & 6 deletions test/index.jl
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,6 @@ inds = Any[1,
NullableArray([1.0]),
NullableArray([:A]),
NullableArray([:A])]
# FIXME: do we want to support this?
# NominalArray([true]),
# NominalArray([1]),
# NominalArray([1.0]),
# NominalArray([:A]),
# NominalArray([:A])]

for ind in inds
if isequal(ind, :A) || ndims(ind) == 0
Expand Down
2 changes: 0 additions & 2 deletions test/indexing.jl
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ module TestIndexing
using DataFrames


# FIXME: this file wasn't included in the tests and didn't pass before the port

#
# DataFrame indexing
#
Expand Down

0 comments on commit beab3a9

Please sign in to comment.