MeteorPackaging / discussions

Ask for Meteor integration help, or discuss Meteor 3rd party library packaging
5 stars 1 forks source link

Fork of lunr.js - wrapped for Meteor #28

Closed lb- closed 2 years ago

lb- commented 9 years ago

Hi,

I am currently maintaining this fork of lunr.js https://github.com/lb-/lunr.js

I submitted a pull request a while ago but reversed it, this wrapper package is available on Atmosphere here: https://atmospherejs.com/lbee/lunr

Might be worth putting on your main list https://github.com/MeteorPackaging or happy to take any suggestions about improvements to what I have changed in my fork.

thanks heaps.

lb- commented 9 years ago

Also, I have been reading more of the main discussion thread here: https://github.com/MeteorCommunity/discussions/issues/14

Should I also set up the lunr namespace on atmosphere? both http://atmospherejs.com/lunr and http://atmospherejs.com/lunrjs are free currently

splendido commented 9 years ago

Yep, reserve them by creating a new organization!

Which repo would you like to see forked under MeteorPackaging?

lb- commented 9 years ago

@splendido I have created the organization: https://atmospherejs.com/lunr I have added @dandv as a member of that organisation also.

The repo I would like to see forked is this one: https://atmospherejs.com/lbee/lunr

If I can then be given access to the repo I will edit it to be in the new organisation namespace & deprecate the old package under my namespace.

Hope I am following the process well :smile:

splendido commented 9 years ago

Hello @lb- ! ...have you by chance followed this discussion?

If you agree on the process, I can create a new repo to start up the wrapping :-)

lb- commented 9 years ago

Do you mean the sister approach, that would make sense as the author has not responded to a previous PR about bringing the meteor files back into their repo https://github.com/olivernn/lunr.js/issues/128 .

Still learning but I understand I will have to manually copy over my meteor components as the git submodule will be created from the original repo: https://github.com/olivernn/lunr.js

splendido commented 9 years ago

yep! ...and provide some tests, in case you still don't have some.

Would you like me to create the new repo? would lunr-wrapper be fine? ...also, are there any other plugin packages around for lunr?

lb- commented 9 years ago

yep of course - I have some tests and will migrate them over + do the other set up (maybe not tonight but when I get to it)

The only other package of this on Atmosphere is an old mrt one: momentjs:moment Cannot find any other plugins for lunr.js only blogs/posts about integrating with different sites.

lb- commented 9 years ago

lunr-wrapper is also fine, makes sense

splendido commented 9 years ago

Ok, see lunr-wrapper (just the repo name, btw...) I've just added you to the lunrjs team.

You can start by copying stuff from the other wrappers linked above.

Please add also meteorpublish and me to the lunr organization :)

lb- commented 9 years ago

Great thanks!

You and meteorpublish have been added to the organisation. will get on it