search

npryce / adr-tools

Command-line tools for working with Architecture Decision Records
Other
4.64k stars 588 forks source link

Fixes for Arch #6

Closed Dockheas23 closed 8 years ago

Dockheas23 commented 8 years ago

Hi, thanks for these tools! I found a couple of problems when running them on Arch Linux.

  1. According to my version of find, the + operator is no longer supported, and the man page recommends to use / instead.
  2. The tests failed to run due to find giving an undefined order of output. Adding a sort fixed it for me.
  3. My team at work have started using these tools, and for our purposes we decided to change the date format to ISO-8601, which prevents any ambiguity that might arise from people in different locations. I'm adding it here in case you'd like to do the same - I realise this one's partially a matter of taste!

I have only tested these on my own machine (Arch Linux). If you want to accept only one or two of the changes, I can split them into separate pull requests if you like.

Dockheas23 commented 8 years ago

Changing + -> / breaks OS-X, apparently. so I'll go back to the drawing board with this one.