Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Now that we have denies, let's @deprecate the any macro #26

Open
gus opened this issue Feb 16, 2011 · 1 comment
Open

Now that we have denies, let's @deprecate the any macro #26

gus opened this issue Feb 16, 2011 · 1 comment

Comments

@gus
Copy link
Member

gus commented Feb 16, 2011

The any and the empty macros are complements and were useful when we only had positive assertions. Now that we have denies we can drop the any macro and use denies.empty where we would have said asserts.any before.

Sugar for sugar's sake is just over-saturation.

@charlietanksley
Copy link
Contributor

I'm working on a patch to deprecate any. I modeled the changes on the deprecated not!. Based on that I assume you want to

  1. Add an @deprecated comment to the Yard documentation for the macro.
  2. Add a deprecation warning for the method.

What isn't clear to me is whether the README should be updated to remove said method. You haven't removed not! from the README, so that tells me to leave it. But I would've thought you would want to remove it, so as to keep from encouraging people to use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants