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

refactor(lodash): Removes most of the lodash dependencies in favor of native js #117

Merged
merged 2 commits into from
Apr 1, 2024

Commits on Apr 1, 2024

  1. refactor(lodash): Removes most of the lodash dependencies in favor of…

    … native js
    
    Most of these lodash dependencies have a native replacement. The only reason they seem to have been used was to wrap the natives with safe checks and returns.
    
    For example the use of `_forEach(list, () => ...)` is a safe version of:
    
    ```
    (list || []).forEach((item) => ...);
    ```
    
    Lodash protects against undefined, null. Others such as _.map also protect by returning an empty map so that chaining is safely allowed, some es6 doesn't allow native.
    acolchado committed Apr 1, 2024
    Configuration menu
    Copy the full SHA
    e41b472 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a609241 View commit details
    Browse the repository at this point in the history