Skip to content

Remove re-exports #31

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

Closed
wants to merge 10 commits into from
Closed

Conversation

nickserv
Copy link
Collaborator

Fixes #19, please refer to it for my rationale.

I removed the dependency on selectorator and some re-exports of Immer and Redux functions. The goal is not to discourage their use, but encourage users to install companion packages separate for more flexible versioning and so they're not locked into the versions bundled with this package. I tried to preserve the docs for the formerly re-exported functions, but I'm not sure about the placement within the readme and wording, so I'm open to suggestions.

@nickserv nickserv requested a review from markerikson April 22, 2018 22:11
@nickserv
Copy link
Collaborator Author

@xDae I saw you upvote #19, I'd appreciate feedback if you have time

@nickserv nickserv removed the request for review from markerikson April 22, 2018 22:12
@nickserv nickserv mentioned this pull request Aug 22, 2018
@markerikson
Copy link
Collaborator

I'm still really not sold on this idea. I think I can agree with not re-exporting the stuff from Redux, because we expect the user to have that installed already, but I really want a selector library included in here.

@nickserv nickserv closed this Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants