Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor: drop lodash #406

Merged
merged 2 commits into from
Feb 18, 2025
Merged

refactor: drop lodash #406

merged 2 commits into from
Feb 18, 2025

Conversation

SukkaW
Copy link
Contributor

@SukkaW SukkaW commented Feb 18, 2025

We don't need an entire lodash library to just escape regexp.

Replace lodash with escape-string-regexp.

All tests passed locally on my machine.

image

Copy link

changeset-bot bot commented Feb 18, 2025

🦋 Changeset detected

Latest commit: e7e4b2b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
eslint-plugin-yml Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Owner

@ota-meshi ota-meshi left a comment

Choose a reason for hiding this comment

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

Thank you!

@ota-meshi ota-meshi merged commit 91d7a0a into ota-meshi:master Feb 18, 2025
9 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants