Skip to content

Commit

Permalink
Recommend against redirecting test output to /dev/null
Browse files Browse the repository at this point in the history
Hiding test output is not that useful: The output may be less cluttered, but when the recipe builds, then no one looks at the Travis log anyway. If it does not build, on the other hand, then crucial error messages may get hidden, as in PR bioconda#3020.
  • Loading branch information
marcelm authored Nov 21, 2016
1 parent 2fb41ab commit 2399e29
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions GUIDELINES.md
Original file line number Diff line number Diff line change
Expand Up @@ -164,8 +164,8 @@ An example of this is [fastq-screen](recipes/fastq-screen).

For command-line tools, running the program with no arguments, checking the
programs version (e.g. with `-v`) or checking the command-line help is
sufficient if doing so returns an exit code 0. Often the output is piped to
`/dev/null` to avoid output during recipe builds.
sufficient if doing so returns an exit code 0. We recommend to not redirect
output to `/dev/null` because it may hide helpful debugging information.

Examples:

Expand Down

0 comments on commit 2399e29

Please sign in to comment.