chore: replace `utils-merge` dependency with `Object.assign()` by jonkoops · Pull Request #138 · pillarjs/router
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 120
Comments
Merged
UlisesGascon merged 1 commit intopillarjs:masterfrom
Dec 20, 2024Merged
chore: replace utils-merge dependency with Object.assign()#138
UlisesGascon merged 1 commit intopillarjs:masterfrom
chore: replace utils-merge dependency with Object.assign()#138
UlisesGascon merged 1 commit intopillarjs:masterfrom
Conversation
Copy link
Contributor
jonkoops
commented
Oct 27, 2024
jonkoops
commented
Replaces the utils-merge dependency with the built-in Object.assign() method.
Copy link
Removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/utils-merge@1.0.1
jonkoops
force-pushed
the
replace-utils-merge
branch
from
108deaf to
8c8a434
Compare
UlisesGascon approved these changes Oct 27, 2024
Copy link
Member
UlisesGascon
left a comment
UlisesGascon
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
bjohansebas approved these changes Dec 2, 2024
Signed-off-by: Jon Koops <jonkoops@gmail.com>
jonkoops
force-pushed
the
replace-utils-merge
branch
from
8c8a434 to
d6058b7
Compare
Copy link
Contributor Author
jonkoops
commented
Dec 2, 2024
jonkoops commented
Dec 2, 2024Did a rebase, the PR should be good to merge again.
UlisesGascon
changed the title
Replace
chore: replace utils-merge dependency with Object.assign()utils-merge dependency with Object.assign()
UlisesGascon
merged commit
8a7c46c
into
pillarjs:master
jonkoops
deleted the
replace-utils-merge
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants