search

TimothyYe / skm

A simple and powerful SSH keys manager
https://timothyye.github.io/skm
MIT License
954 stars 54 forks source link

Fix empty skm directory #6

Closed lsjurczak closed 6 years ago

lsjurczak commented 6 years ago

My solution is to check on init if skm directory is empty and remove it.

Maybe you have another way to solve this. I am open to suggestions.

Closes #5

TimothyYe commented 6 years ago

@lsjurczak Thanks for your PR, I reviewed it.

At line 20 of actions.go:

color.Red("%sFailed to remove existing empty key store!", skm.CheckSymbol)

To output error message, I think we can use skm.CrossSymbol instead. Would you please update the PR, then I'll merge it, thanks!

TimothyYe commented 6 years ago

@lsjurczak

Thanks for your contribution, this fix will be included in release V0.3.2