hound
hound copied to clipboard
Override default directory for (or pass dir to) take_screenshot/1
It would be nice to have a little more control over where screenshots get written. There are two options I'd like to have (either or both would be great):
- Set a default directory location in the config
- Pass a directory to
take_screenshot/1
Both of these would ideally retain the existing name generation feature screenshot-#{year}-#{month}-#{day}-#{hour}-#{minutes}-#{seconds}.png
.
I'm happy to submit a PR if this is something you're interested in. Option 2 I could do fairly easily, option 1 I'd need to poke around the code base a bit more first. Thanks!
Hi, Someone started to work on this functionality but we were missing tests and the PR is still pending: https://github.com/HashNuke/hound/pull/163 If you would like to give it a try, I would be glad to merge it. Thank you very much.
Sure, happy to pick up where they left off. Is it easiest if I just submit it as a new PR (I don't know if there's another option tbh)?
@tuvistavie are there other methods that could make use of :temp_dir
? If not, it might be clearer to use :screenshot_dir
or something to make it more explicit that this value will change where screenshots are stored. Thanks!