-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
Maintenance #2819
Maintenance #2819
Conversation
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.
LGTM. Great 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.
Looks good to me, left a comment we might want to fix before v5 release?
@@ -10,22 +10,13 @@ const selectGlobal = state => state.global || initialState; | |||
const selectRouter = state => state.router; | |||
|
|||
const makeSelectCurrentUser = () => |
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.
As per #2806 (comment), do we want to change some of these selector factories to regular selectors?
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.
Yes it's in the to do list for #2812. The changes you see here are just because prettier wanted them.
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. |
React Boilerplate
react
to v16.12 yet as@hot-loader/react-dom
is still on v16.11 but will make sure to do it before we release v5.0.I ran our full regression tests on this. Everything should be working.