girzel / ebdb

An EIEIO port of BBDB, Emacs' contact-management package
67 stars 11 forks source link

Add ebdb-complete.el #32

Closed tumashu closed 7 years ago

tumashu commented 7 years ago

今天有点时间,我把bbdb-handy port过来了,handy 这个名字没有什么特殊的含义,如果你有更好的名字,可以换掉.

girzel commented 7 years ago

嗯,谢谢!我得很惭愧地承认,这是我第一次接受 pull request,我稍微研究一下再处理。

tumashu commented 7 years ago

好的

girzel commented 7 years ago

我确实想换一个名字,但还没想好。其实,EBDB整个completion和insertion的机制应该重构一下,像ebdb-handy、company-ebdb,helm-ebdb,都是一个功能:选一个或多个记录,然后在当前的buffer里加那些记录的一些信息。我还写了一个ebdb-cite-records-mail,差不多一样,只不过它可以在org-mode或别的mode insert适合当前的major-mode的文字。但其实都是一个意思,我要想办法把这些机制稍微统一一下。

girzel commented 7 years ago

要不干脆叫ebdb-complete?然后我以后就慢慢把这些不同的功能合并,让用户选他们想用的completion方法。

tumashu commented 7 years ago

叫ebdb-complete听不错,统一一下是最好了,不然后期维护啰嗦

tumashu commented 7 years ago

这个pull request怎么处理,你准备直接统一,还是先合并,后续慢慢规整,如果要合并,我就更新一下词法作用域

girzel commented 7 years ago

Pull request 你应该可以从那头再加commit,然后我这边接受时把它们squash就好了。你把package name该一下,还有lexical binding 那个,我就接受,然后再做修改。

tumashu commented 7 years ago

package名称改成ebdb-complete?

tumashu commented 7 years ago

OK