I can't approve it, but I have two suggestions: First, The istarget test should be more like "if not target, exit" so that it doesn't affect the formatting of the rest of the file. Second, it's better to use a find-type command than to explicitly list all the tests. Makes it easier to add new tests later.