search

ruby / tmpdir

Retrieve temporary directory path
5 stars 11 forks source link

more expressive implementation by using "meaningful" variable names that reflect the intent #21

Open pvdb opened 2 years ago

pvdb commented 2 years ago

Thanks for accepting #16, @nobu, and for fixing #17 ... much appreciated! :blush:

Here's another quick refactor for your consideration... not a functional change, just a style change, but hopefully you like!

Note that I considered making both candidate_dirs and fallback_dirs class variables - similar to @@systmpdir - but that would require a very significant refactor of the unit tests, so I've not done that in this PR.