Update primary error functions to take a context, deprecate old functions

[schristoff]

No description provided.

[louisruch]

This looks really good! I have one comment about Wrap calling EDeprecated
The only other general comment I would have is that all the test files only test EDeprecated, perhaps if we do add an unexported e that could be used instead...I will look again Monday great work so far!

[jimlambrt]

If we are planning to pass a context to Wrap in the future we should also make a WrapDeprecated function now; if we are not planning to pass the context to Wrap, we won’t be able to call E because we don’t have a context in the future. Similar comment on Convert.

+1 on this idea.

[jimlambrt]

Amazing work! I just have a few comments/suggestions.

[jefferai]

There are a large number of cases in this PR where there is a context in the function but EDeprecated is used instead of just using the context passed in. It would be good to go through and use the context when available.

[louisruch]

The WithoutEvent and using context.Todo works well and neatly resolves my concerns around tying us down to never being able to deprecate these functions. ty