-
Notifications
You must be signed in to change notification settings - Fork 5.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
[webapp] Remove underscore #12238
base: devel
Are you sure you want to change the base?
[webapp] Remove underscore #12238
Conversation
Hey @harryadel, awesome to see you here! |
Sure, @harryadel, I'll look carefully at them and adjust what is needed. Again thanks a lot for the care and time spent making the PRs! |
I think the underscore PRs might use a second look now @StorytellerCZ @Grubba27 |
Sure! I will add them to my to-do list for the next weeks. I plan on reviewing them again and maybe working on a new release for the 2.x track with these changes as long as in the 3.x track as well |
This will be the very first PR of many to come in order to purge Underscore from Meteor codebase.
I must admit @filipenevola was right and I was wrong, splitting this endeavor in multiple mini-PRs is the way to go #11869 ran into a deadlock and I couldn't figure out why it was failing, just because it worked for Blaze doesn't mean it's ought to work for Meteor so credit where it's due.
One last thing, I don't have much time available so once the PRs are created whomever is reviewing these PRs please go ahead and modify them as you see fit in order to get them merged, thanks!