Skip to content

Remove lodash #555

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
Merged
gustavohenke merged 9 commits into from
Jul 12, 2025
Merged

Remove lodash #555

gustavohenke merged 9 commits into from
Jul 12, 2025

Conversation

@gustavohenke
Copy link
Member

@gustavohenke gustavohenke commented Jul 12, 2025

  • Functions with native drop-in or simple-enough-alternative (e.g. sortBy) have been replaced with the native version;
  • Functions simple to maintain and used a lot have been bundled in an utils.ts file with a bespoke version;
  • Functions not as simple to maintain and not used much have been migrated inline, e.g. _.get() or _.defaults()

@coveralls
Copy link

coveralls commented Jul 12, 2025

Coverage Status

coverage: 97.586% (-0.2%) from 97.738%
when pulling e7dca34 on bye-lodash
into a7b6174 on main.

@gustavohenke gustavohenke merged commit 7437b0f into main Jul 12, 2025
21 checks passed
@gustavohenke gustavohenke deleted the bye-lodash branch July 12, 2025 13:59
@gustavohenke gustavohenke mentioned this pull request Jul 12, 2025
Closed
@dynst dynst mentioned this pull request Jul 12, 2025
Closed
gustavohenke added a commit that referenced this pull request Aug 3, 2025
@gustavohenke
Fix bug with hide: 0
37212b7 
Instead of hiding the output from command 0,
this would actually not hide any output.

Introduced by #555
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gustavohenke @coveralls