Code review comment for lp:~sseman/workspace-runner/download-logs

Revision history for this message
Aaron Bentley (abentley) wrote :

I can live with the functionality, but I don't think "logs" is the right word, because
1. it's downloading directories containing logs, not just logs.
2. it can download arbitrary directories, not just those containing logs.

I suggest "download-dir".

Please add an example yaml file.

review: Needs Fixing

« Back to merge proposal