Meteor-Community-Packages / meteor-collection-hooks

Meteor Collection Hooks
https://atmospherejs.com/matb33/collection-hooks
MIT License
658 stars 90 forks source link

Remove underscore + modernize code base #248

Closed sebakerckhof closed 4 years ago

sebakerckhof commented 5 years ago

Continuation of: https://github.com/matb33/meteor-collection-hooks/pull/247 with a broader scope to fully modernize the code base. All tests are passing except in direct.js i had to disable some lines for which I think is a transpilation bug.

Note that I did remove some BC stuff for really old versions of Meteor (pre 0.9)

sebakerckhof commented 5 years ago

ping @sakulstra

sebakerckhof commented 5 years ago

ping @zimme

StorytellerCZ commented 4 years ago

@sebakerckhof OK, the plan is to merge the other two PRs and Meteor 1.6.1 branch, release and then update this and release as 1.0.

StorytellerCZ commented 4 years ago

@sebakerckhof Alright, v0.9.1 has been released. Time to get this moving again.

StorytellerCZ commented 4 years ago

Did the update, please check that I haven't done some mistakes in merger. @sebakerckhof Thanks!

StorytellerCZ commented 4 years ago

Will have to update lint rules to take care of new features. But I would like to wait on this till we decide in the organization about what rules to use.