Added a helper success case for Result<Void, Error> types

[bojanstef]

This change allows the following:

func foo(result: (Result<Void, Error>) -> ()) {
+  result(.success)
-  result(.success(()))
}

[theblixguy]

cc @jshier

[jshier]

I'd like to see this addition, as Result<Void, Failure> is fairly common and it feels awkward to keep having to write .success(()). However, I'm pretty sure that additions to the APIs of standard library types have to go through the evolution process. I have my own ideas for additions (tryMap, perhaps fold), so we may be able to combine all of the suggestions into a single pitch and proposal, but I won't have time to do it myself for a bit.

[bojanstef]

Thanks @jshier, I was hoping I could squeeze this change in but if it has to go through the evolution process then so be it 😄

In the meantime, should I close this PR and start writing a proposal?

[hfhbd]

Hey @bojanstef,
did you create a new pitch, to improve the Result type? If not, @jshier`s ideas and maybe my idea (#27908) could also be included.

[CodaFi]

@bojanstef Any movement here?

[bojanstef]

@bojanstef Any movement here?

I haven’t done anything.

[shahmishal]

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

• How to change the base branch: (Link)
• More detail about the branch update: (Link)