bflad / chef-stash

Chef Cookbook for Atlassian Stash
Other
37 stars 42 forks source link

Moved checksum's and download url calculation into helper.rb #169

Closed katbyte closed 7 years ago

katbyte commented 8 years ago

This allows calling cookbooks to change the version on the fly.

Followed the pattern in:

Issue #167

katbyte commented 8 years ago

@linc01n I noticed you merged the versions PR. Rebased this off that branch so there would be no merge conflict. If you accept this one i also have the newest version checksums added and can create a PR for them aswell:

https://github.com/katbyte/chef-stash/tree/versions.201610

iluxame commented 7 years ago

@katbyte we also waiting for this PR. Do you intend to fix coding style stuff in order to pass CI?

katbyte commented 7 years ago

@iluxame I have rebased this PR off the latest master, however the errors causing the CI build to fail do not seem related to my changes (but i could be wrong)

screen shot 2017-05-30 at 16 13 17

@linc01n Is there a reason this PR was not merged? Is there anything I can improve/fix?

linc01n commented 7 years ago

Sorry for the late reply. What I am concerned is moving this out of the attribute.rb is that the right way to do. I asked this question to some chef people when I was attending a workshop. He said it should be fine.

Going to merge this PR now.

Thanks!

linc01n commented 7 years ago

Fixed the rubocop and foodcritic error. Changed the helper a bit in #182