search

AArnott / Validation

Method input validation and runtime checks that report errors or throw exceptions when failures are detected.
Microsoft Public License
131 stars 22 forks source link

Add obsolete warnings #51

Closed scottdorman closed 3 years ago

scottdorman commented 3 years ago

We should [Obsolete] the ValidState API with a message telling folks to use Verify.Operation instead.

That is similarly redundant with Requires.Argument and should be deprecated. Having redundant methods for the sake of sounding "fluent" makes an API more confusing, IMO.

Originally posted by @AArnott in https://github.com/AArnott/Validation/issues/49#issuecomment-701453387

scottdorman commented 3 years ago

Should the unit tests be removed for ValidState and That or should they be wrapped in a pragma to ignore the CS0618 warning?