mavenlink / brainstem-js

Brainstem API adapter for Backbone complete with relational models
MIT License
20 stars 2 forks source link

Prefer Object.assign to $.extend to merge objects #209

Closed guanw88 closed 7 months ago

guanw88 commented 7 months ago

Although Object.assign and $.extend have some differences (https://stackoverflow.com/questions/38345937/object-assign-vs-extend), the should not apply to us, as 1) we do not use the deep: true option in these methods, and 2) we are always merging in an empty object ({}), so it should never be undefined.