I think the each loop is fine (although meant to comment that I'd like a space after the comma for readability `|exe, source_url|`).  I dislike that `node['travis_buddy']['install_action']` is used for both.  Seems contrived, unless you anticipate people wanting to set it to `:delete`.  Come to think of it, if that's the reason, I'm fine with it.