if we want to do this ...

[emacs-citar]

if we want to do this ...

[mtreca]

Hi @bdarcus, thanks a lot for the invitation. I am interested in collaborating for this package. I am still pretty busy at work, but I will probably take half a day this week to catch up on different threads regarding the project and think about how I could contribute best to this project. Thanks for your involvement!

[mtreca]

Hi again. I sadly still do not really have time on the project, but it seems that my help is not really needed at this point. Stellar work on this package! The elisp looks really clean and the whole integration with completion-read and the emabrk ecosystem is very elegant. I will be happy to test it and provide feedback.

[bdarcus]

Sounds good.

It looks like it will be a couple of weeks before Titus can look at our PRs, so that gives you some time too.

The key changes I made on my code are on the key --read functions.

The embark author also helped me optimize the completing-read function.

I ended up merging the two code bases here in the repo, as it turned out to be easy.

The more I think about it, I think there's big value in integrating the change directly into bibtex-completion, and I think I prefer that.

But if not, there's little benefit in hosting a separate package there, and significant downside.

Doing it here, however, gives us flexibility to make any changes at our own pace; not just to the code, but also documentation, etc.

And I expect as selectrum et al take off, there will be more people interested in this sort of solution, and so possibility for helpful PRs, etc.

[bdarcus]

I've decided to focus on this, more than the PR, as stuff can always be moved back, and it's just much easier to manage with a separate repo, issue tracker, etc., and without having to worry about legacy stuff.

Plus I setup the CLI to warn me if any of the elisp was problematic, which it does well.

I have a couple of open issues and PRs @mtreca, if you have any feedback.

[bdarcus]

Yeah, to bring you up-to-speed, main changes I made was to remove the macro (for simplicity and flexibility), and to switch to completing-read-multiple. Also, changed map to cl-loop.

[bdarcus]

Also, for what I'm working on now, see #37.