-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
R4R: Consistent Operators #3679
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3679 +/- ##
==========================================
- Coverage 61.21% 61.2% -0.02%
==========================================
Files 190 190
Lines 13995 13995
==========================================
- Hits 8567 8565 -2
- Misses 4892 4894 +2
Partials 536 536 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing the dirty work 💪
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please hold this off - I'm happy to incorporate these changes in #3678. If this is approved it will make my branch stale and will give me quite some pain
@alessio - would you like me push the relevant changes to your branch? It will literally take me 1 minute |
Co-Authored-By: rigelrozanski <rigel.rozanski@gmail.com>
@rigelrozanski that'd be greatly appreciated! |
closes #2360
Find replace changes made to go files:
Post replace fixes:
docs/
)PENDING.md
with issue #Files changed
in the github PR explorerFor Admin Use: