-
Notifications
You must be signed in to change notification settings - Fork 992
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
Purge <- usage when not strictly necessary #3590
Comments
IMO this is not really that important stuff. I prefer to use |
I don't really notice much myself, but the linked comment and this commit: 5024343 led me to file this in the spirit of consistency. Added |
Like @jangorecki, I don't think I would have put effort into this either. But now Michael has, it's quite nice to get this out of the way; it's higher value than I thought. It should make PRs easier/faster, particularly as we see new contributors hopefully. |
Follow-up to comment here:
#3582 (comment)
Also added to Contributing
There's a decent amount of manual clean-up to do, unless someone has as better regex for catching the valid
<-
usages:I wonder how feasible it is to just blanket replace & then patch in the
=
that cause errors because<-
is required 🤔The text was updated successfully, but these errors were encountered: