Suggestion for hint: wrap in Some

[Lucretiel]

Consider this error:

"Expected Option<T>, found T"

A useful hint/suggestion would be to propose wrapping the erroring expression in Some

[detrumi]

Rustc does suggest to use Some, but we're currently not showing the suggestion:

help: try using a variant of the expected enum: `Some(...)`

[jonas-schievink]

We now show the rustc hint, and also have a builtin diagnostic + quickfix when this happens in a function's return value

[eminence]

This now mostly works, but there's a small problem when you have serde in scope:

[Veykril]

The first diagnostic comes from rustc, the Wrap in Some is the diagnostic from rust-analyzer which should hopefully work correctly.

[Veykril]

We can actually close this now that wrap in some is a proper type mismatch diagnostic I believe.

[eminence]

The first diagnostic comes from rustc

Ahh, you're right, thanks. This is currently tracked at rust-lang/rust#26454