Re-implements "retagging" action in elisp to get around sed version inconsistencies between platforms #158

Merged
merged 5 commits into from Mar 17, 2013

Conversation

Projects
None yet
2 participants
Contributor

dlobraico commented Mar 11, 2013

As discussed in issue #150 and on mu-discuss. I didn't see an elisp style notes anywhere. I did add two small helpers but chose not to prefix them with mu4e because I don't see them being used by end-users. I'm not sure if that was the right way to go, so let me know and I can change it.

Owner

djcb commented Mar 16, 2013

Looks good to me (though I don't use tagging myself) -- but could you prefix you helper-functions with mu4e~ to be consistent with the rest? Thanks!

Contributor

dlobraico commented Mar 16, 2013

How's that?

Thanks!

djcb merged commit ae1deb4 into djcb:master Mar 17, 2013

Owner

djcb commented Mar 17, 2013

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment