Nice! I left a question on one line of the commit.

Also, could you update the README as part of this PR with some examples? (the readme examples are run as tests with Midje so they serve two purposes: documentation and additional tests).

Thanks.

Many thanks! 👍

BTW, the test instructions were correct as far as I can tell. If you rm test/readme.clj then run lein test-all, the readme test is regenerated and run automatically as part of lein test-all:

(! 610)-> rm test/readme.clj 

Thu Oct 11 14:36:29
(sean)-(jobs:0)-(/Developer/workspace/clj-time)
(! 611)-> lein test-all
Performing task 'test' with profile(s): 'dev,spec,base,system,user,provided,midje'
... lots of warnings snipped ...
lein test clj-time.coerce-test

lein test clj-time.core-test

lein test clj-time.format-test

lein test clj-time.inst-test

lein test clj-time.jdbc-test

lein test clj-time.local-test

lein test clj-time.periodic-test

lein test clj-time.predicates-test

lein test clj-time.spec-test

lein test clj-time.types-test

lein test readme

Ran 127 tests containing 622 assertions.
0 failures, 0 errors.

Can you confirm you get that behavior without running lein test-readme

Yes, that is what I am getting. I also cleaned up the readme to be slightly more concise, and more consistent with the rest of the text.

Since I already merged the PR, feel free to send any further updates in a separate PR. I'm updating the change log and plan to cut 0.14.6 later today with these additions.

I removed the explicit call to test-readme in the instructions. If I run it I get:

> lein test-readme
'test-readme' is not a task. See 'lein help'.

But a plain lein test picks up readme.clj and runs it.

Ok, I will follow up with another. Sorry for the extra work. I need github to do a convincing preview of the markdown for me. Much appreciated!