Skip to content
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

Update dependencies #2832

Merged
merged 1 commit into from
May 26, 2019
Merged

Update dependencies #2832

merged 1 commit into from
May 26, 2019

Conversation

kibertoad
Copy link
Contributor

No description provided.

@Bradcomp
Copy link
Member

Not to derail, but one of our dev dependencies is an old version of Ramda... That seems troublesome. I think we should look into that.

@CrossEye
Copy link
Member

@Bradcomp:

Yes we should. I'm pretty sure that we can just replace it with @-latest. We use Ramda in the build. If our build can't use a recent version of our library, either the library or the build should get some serious attention.

But as you said, that doesn't need to derail this PR. It looks good to me. If you agree, then I think we can go ahead and merge it.

@Bradcomp
Copy link
Member

:bowtie:

@Bradcomp Bradcomp merged commit ed191e6 into ramda:master May 26, 2019
@CrossEye CrossEye mentioned this pull request Feb 12, 2020
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.

3 participants