Skip to content
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

update asset_versions.yml when only client changes #96

Closed
wants to merge 2 commits into from

Conversation

wacii
Copy link
Contributor

@wacii wacii commented Jun 8, 2015

asset_versions.yml was only being updated when the drop-in assets were updated.

Forgot to ask this before, but should I document how to update the client assets? Like add a note to the readme or something?

@@ -0,0 +1,30 @@
require 'yaml'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.


def save
if @asset_versions.dropin_version == @version
puts 'Dropin assets up to date'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not write to stdout. Use Rails' logger if you want to log.

@wacii wacii closed this Dec 20, 2015
@wacii wacii deleted the revise-tasks branch December 20, 2015 22:16
@wacii wacii restored the revise-tasks branch December 20, 2015 22:16
@wacii wacii deleted the revise-tasks branch December 20, 2015 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants