search

mozillazg / rust-pinyin

汉字转拼音
https://docs.rs/pinyin/
MIT License
203 stars 24 forks source link

使用 cargo-tarpaulin 的二进制 #37

Closed upsuper closed 5 years ago

upsuper commented 5 years ago

每次编译太没有效率了……

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 894430f2570def016b69a313025ada7e77fa59f2 on upsuper:tarpaulin-binary into 51cb89294924939b864bf24b471081cc99caa61d on mozillazg:master.

upsuper commented 5 years ago

@LuoZijun 建议先合并这个。 #36 去掉了cargo cache，因为现在这个库没有任何依赖，cache并没有意义。然而这导致tarpaulin编译拖慢很多。

LuoZijun commented 5 years ago

@upsuper 这个改动看起来相当好，但是我不知道 @mozillazg 是否会介意我合并。 @mozillazg 提到近来没有怎么碰 Rust 了，所以有时候找我看下。但合并一般都是他来的 :)

mozillazg commented 5 years ago

@LuoZijun 我不介意呀，所有 PR 你觉得可以合并的话，合并就好，相信你的判断。

LuoZijun commented 5 years ago

@mozillazg Ok :)