Skip to content

Conversation

@FrederikBolding
Copy link
Member

We already have lodash in the dependency tree of most of our repositories. Therefore we prefer using that for utils as well. This also reduces the amount of patches we have to create to get lodash running in the clients.

Closes #252

@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​types/​lodash@​4.17.19 ⏵ 4.17.2010010078 +191 -1100

View full report

@FrederikBolding FrederikBolding enabled auto-merge (squash) July 10, 2025 07:56
Copy link
Contributor

@cryptodev-2s cryptodev-2s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@FrederikBolding FrederikBolding merged commit e528b1f into main Jul 10, 2025
22 checks passed
@FrederikBolding FrederikBolding deleted the fb/use-full-lodash branch July 10, 2025 08:04
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.

Use lodash rather than lodash.memoize

3 participants