ExtCore
ExtCore copied to clipboard
Both Result and Choice for error handling
- Forked individual files from #31 and reverted original files
- Added
Compatibilitynamespaces and moved someChoiceconstructs into these - Renamed
Notification.ErrortoNotification.Exceptionin order to avoid confusing things withResult<,>
Note that I've added same kind of test as can be found for Fold for FoldBack @jack-pappas
@vasily-kirichenko and @jack-pappas thoughts? Does it look ok?
Thanks @wallymathieu — I’ll take a look at this over the weekend.
@jack-pappas could you please publish a new NuGet package after merging this PR?
https://gist.github.com/wallymathieu/ad8fe3d5b85875cf6f487ad0180be660 ?
Should we perhaps try to split this pull request into more granular parts? A lot changes are mostly copy and modify (in order to be compatible). @vasily-kirichenko do you have an idea of how this could be done?
Is there a plan to merge this?
Should we close this pull request @jack-pappas (as it is to big) ?
I'm closing this PR as it is probably to big to review or accept. Best of luck!
@jack-pappas Please, merge this PR. I had to build an ExtCore package myself and publish it on a private server, everything's been working OK.
Since @vasily-kirichenko says that it's working fine, I've reopened the pull request
Yes, it's working 100% fine.
Then perhaps move this PR to point to another branch on this repository in order to iterate on it?
@wallymathieu Could you merge master into it to make AppVeyor happy?
Will do!