Skip to content
This repository was archived by the owner on Jan 10, 2023. It is now read-only.

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

Merged
linc01n merged 1 commit intobflad:masterfrom
katbyte:#167
Jun 13, 2017
Merged

Moved checksum's and download url calculation into helper.rb #169
linc01n merged 1 commit intobflad:masterfrom
katbyte:#167

Conversation

@katbyte
Copy link
Copy Markdown
Contributor

@katbyte katbyte commented Aug 9, 2016

@katbyte
Copy link
Copy Markdown
Contributor Author

katbyte commented Nov 7, 2016

@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
Copy link
Copy Markdown

iluxame commented Mar 8, 2017

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

@katbyte
Copy link
Copy Markdown
Contributor Author

katbyte commented May 30, 2017

@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
Copy link
Copy Markdown
Collaborator

linc01n commented Jun 13, 2017

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 linc01n merged commit 99142ae into bflad:master Jun 13, 2017
@linc01n
Copy link
Copy Markdown
Collaborator

linc01n commented Jun 13, 2017

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants