Upper bound Adapt to avoid test failure.

[andreasnoack]

@jishnub would you be able to review?

[jishnub]

Perhaps it'll be better to update the failing test here, instead of restricting the version? I think we can change

StructArrays.jl/test/runtests.jl

Line 1194 in 8fcd8d3

Adapt.adapt_storage(::ArrayConverter, xs::AbstractArray) = convert(Array, xs)

to use Adapt.adapt_structure instead of Adapt.adapt_storage? The following method appears to be narrower than the current one, and works on both versions:

Adapt.adapt_structure(::ArrayConverter, xs::UnitRange) = convert(Array, xs)

It is also closer in intent to what is being tested.

[jishnub]

Rebasing on master should fix the invalidation test failure

[andreasnoack]

I forgot to answer here. It wasn't immediately obvious to me what the tests were intended to cover so I didn't feel comfortable adjusting them but if you believe your change above aligns with the original intent of the tests then I think we should go with that instead of this PR.

[jishnub]

@aplavin might be able to suggest, as I think he's familiar with Adapt. In general, Adapt.adapt_structure appears to be the recommended route instead of the lower-level Adapt.adapt_storage

[aplavin]

Sorry, I don't really work with Adapt, have no idea about its functions.

[jishnub]

Is this unnecessary now after #317?

[andreasnoack]

I think so