-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add cuttle cask #10184
Add cuttle cask #10184
Conversation
@LesleyMdCampo looks like we need an empty line at the end of the file for the test to pass |
version '1.1' | ||
sha256 '7133beb24c3a37fe13f12d0b8110f1fcaf541d80e938cfaafb37a4d47bfcca15' | ||
|
||
url 'https://github.com/oakmac/cuttle/releases/download/v1.1/cuttle-v1.1-mac.dmg' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you use #{version} here. i.e download/v#{version}/cuttle-v#{version}-mac.dmg
Once you've made the changes can you please also squash your commits? |
@fanquake - I'm not sure what the error in the Travis CI build is here. Can you help? |
version '1.1' | ||
sha256 '7133beb24c3a37fe13f12d0b8110f1fcaf541d80e938cfaafb37a4d47bfcca15' | ||
|
||
url 'https://github.com/oakmac/cuttle/releases/download/v#{version}/cuttle-v#{version}-mac.dmg' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The error is here. You’re using interpolation (#{version}
), so you need to use double quotes, or else it’s taking the whole string literally.
@LesleyMdCampo Thanks for updating, looks good now. |
No description provided.