-
Notifications
You must be signed in to change notification settings - Fork 3k
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
fix(operators): export reserved name operators on prototype #1927
Conversation
cc @IgorMinar |
Had an extraneous semicolon that was failing the build, so I removed it. Used it as an opportunity to try out the new ability for maintainers to modify submitted PR branches directly (even Although I don't like this solution, I can't think of any other one. This whole underscore business is unfortunate to begin with. So that said, I think this is acceptable. I'll wait a few days to see if anyone wants to object? |
LGTM |
Needs rebase tho |
rebased as #1932 |
@IgorMinar I rebased this already, is that the only change? |
@jayphelps I just rebased. no other changes. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
closes #1924