Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify Vararg/UnionAll deprecation warning #49558

Merged
merged 1 commit into from
Apr 29, 2023
Merged

Conversation

timholy
Copy link
Member

@timholy timholy commented Apr 28, 2023

Fixes #49553

@aviatesk aviatesk merged commit 219dc10 into master Apr 29, 2023
@aviatesk aviatesk deleted the teh/va_unionall_warning branch April 29, 2023 01:21
KristofferC pushed a commit that referenced this pull request Dec 10, 2024
This warning message
```
WARNING: Wrapping `Vararg` directly in UnionAll is deprecated (wrap the tuple instead).
You may need to write `f(x::Vararg{T})` rather than `f(x::Vararg{<:T})` or `f(x::Vararg{T}) where T` instead of `f(x::Vararg{T} where T)`.
```
(last extended in #49558) seems clear enough if you wrote the code. But
if it's coming from 10 packages deep, there's no information to track
down the origin.

Turns out you can do this with `--depwarn=error`. But the message
doesn't tell you that, and doesn't call itself a deprecation warning at
all.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spurious deprecation warning from Vararg !<: Type
3 participants