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.

Sign up for GitHub

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

Jump to bottom

chore(ramda): Remove dependency on external library #222

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chore(ramda): Remove dependency on external library #222

wants to merge 1 commit into from

Commits on Dec 22, 2021

  1. chore(ramda): Remove dependency on external library 

    *TLDR* Reduce final module size and dependency clutter.

Ramda is cool. Using it can help make code more functional and
expressive. However, it’s not always a good idea to couple a library
to a third party dependency if we can solve the problem with a few
lines of code.

I guess this wouldn't be so much of an issue if the build output
something which did not contain the `require` for the ES module.
The main issue with requiring this third party dependency is that
it increases the amount of code that needs to be downloaded, since
we have another library which is required from this one. This is
mainly a problem for those of us who are using native ES modules
rather than using bundlers with tree shaking I suppose.
    @NeoVance
    NeoVance committed Dec 22, 2021
    Configuration menu
    Copy the full SHA
    e4f7285 View commit details
    Browse the repository at this point in the history