-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Remove $maxDepth attribute, parameter, etc. #3336
Comments
Also replaces #3180. |
Removing the The only way to clean up All these optional parameters should have never been added, they make for a terrible API. I am not sure, though, if the benefit of cleaning up |
The only parameter that I used sometimes is the ignore case one. Never used any other. Migrating to a dedicated assertion for that seems a reasonable upgrade path for me. |
Superseded by #3341. |
Moved here from #3335 (comment).
The text was updated successfully, but these errors were encountered: