Skip to content
This repository has been archived by the owner on Aug 18, 2021. It is now read-only.

Remove the lodash.assign dependency #393

Merged
merged 1 commit into from Sep 27, 2016
Merged

Conversation

fson
Copy link
Contributor

@fson fson commented Sep 25, 2016

lodash.assign is deprecated:

npm WARN deprecated lodash.assign@4.2.0: This package is deprecated. Use Object.assign.

@hzoo
Copy link
Member

hzoo commented Sep 27, 2016

We were doing this as part of https://github.com/babel/babel-eslint/pull/361/files but we can merge this one. can you fix the merge conflict?

lodash.assign is deprecated:
```
npm WARN deprecated lodash.assign@4.2.0: This package is deprecated.
Use Object.assign.
```
@fson
Copy link
Contributor Author

fson commented Sep 27, 2016

Thanks! Fixed.

@hzoo hzoo merged commit 4cf9ac1 into babel:master Sep 27, 2016

// monkeypatch estraverse-fb (only for eslint < 2.3.0)
try {
var estraverseFb = eslintMod.require("estraverse-fb");
estraverses.push(estraverseFb);
assign(estraverseFb.VisitorKeys, t.VISITOR_KEYS);
Object.assign(estraverseFb.VisitorKeys, t.VISITOR_KEYS);
Copy link
Member

Choose a reason for hiding this comment

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

👍

nicolo-ribaudo pushed a commit to babel/babel that referenced this pull request Nov 14, 2019
lodash.assign is deprecated:
```
npm WARN deprecated lodash.assign@4.2.0: This package is deprecated.
Use Object.assign.
```
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants