push! type piracy

[adienes]

SentinelArrays.jl/src/BufferedVectors.jl

Line 32 in fa840f9

@inline Base.push!(buffer::T, x::S) where {T,S} = push!(buffer, convert(T, x))

I think the piracy here should be avoided if possible. I noticed this when the error message for something like push!(0, 0) changed

[ancapdev]

I hit this today. To type pirate an function as core as Base.push!() seems rather presumptuous.

[quinnj]

@Drvi, would you mind fixing this? Otherwise, I can try to fix today

[Drvi]

Ah I see what is going on, I'll make a PR shortly. Thanks all!