search

cirho / powerline-rust

powerline-shell written in Rust
MIT License
50 stars 25 forks source link

Add time module #15

Closed inamiy closed 4 years ago

inamiy commented 4 years ago

Hi 👋 I just came up with your nice powerline impl in Rust, but it was missing important time.rs, so I added it in this PR.

cirho commented 4 years ago

Looks nice, but not many people add current time to shell prompt. That's why, I would like to have time segment as opt-in feature. Although chrono is not big, it's still additional dependency.

Could you put time segment and chrono behind feature and move time segment from main binary file to maybe new example?

If you do this I will gladly merge it.

inamiy commented 4 years ago

147c229 Moved as time feature that depends on chrono.

cirho commented 4 years ago

Thanks 😃