ndrwrbgs
ndrwrbgs
## Paths _make sure work here includes thoughts about how to handle the case where Directory method is called but a File exists at the path_ - [ ] DirectoryExists...
It looks like #228 added what I was wanting, but that the latest of the https://www.nuget.org/packages/CacheManager.SystemRuntimeCaching was published the year prior.
Fixes #303 A little less "standard" than a wrapping exception, but satisfies the requirements without functional change beyond the stack trace. Does lose the stack trace of BaseCacheManager:L130, instead doing...
Fixes #303 Breaking change in that the exception type thrown to the caller will not be the same. May not need a major version bump though since it's only a...
My largest concerns I foresee and defer to @MichaCo on the changes that should/should not be made: - [ ] Deviation from current centralized version - [ ] CLS Compliance,...
There are ways to 'rethrow' the exception directly rather than throw ex again (which will overwrite the StackTrace and says that the exception came from BaseCacheManager rather than the underlying...
Since having that argument means the handle requires a serializer for sure (rather than if it, say, read from the `configuration`), we could provide the same error visibility for handles...
**Is your feature request related to a specific problem? Or an existing feature? Please describe.** Specific problem. A user can call `Policy.Execute` on a `Func` without any signal they've done...
I got worried by the nuget page showing last publish in 2019, but it looks like there's been ongoing development in master. Is the package now being shipped somewhere other...
** NOT SAFE AS IS ** _This review is mostly to start the discussion and get a feel for how open the maintainers are to breaking changes_ # Problem During...