Skip to content
This repository has been archived by the owner on Apr 27, 2021. It is now read-only.

Commit

Permalink
Merge pull request #155 from jdlich/delete_remote_files
Browse files Browse the repository at this point in the history
Delete remote files
  • Loading branch information
csaunders committed Jun 9, 2015
2 parents ac59a54 + d6539a3 commit e48b45f
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions lib/shopify_theme/cli.rb
Original file line number Diff line number Diff line change
Expand Up @@ -150,10 +150,9 @@ def watch
watcher do |filename, event|
filename = filename.gsub("#{Dir.pwd}/", '')

next unless local_assets_list.include?(filename)
action = if [:changed, :new].include?(event)
action = if [:changed, :new].include?(event) && local_assets_list.include?(filename)
:send_asset
elsif event == :delete
elsif event == :delete && !options['keep_files']
:delete_asset
else
raise NotImplementedError, "Unknown event -- #{event} -- #{filename}"
Expand Down

1 comment on commit e48b45f

@jdlich
Copy link
Contributor

@jdlich jdlich commented on e48b45f Aug 24, 2015

Choose a reason for hiding this comment

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

@csaunders FYI, there's a problem with this change. It fixes the remote delete issue but the watch fails randomly now because of an unkown event error. I'll have to take another shot at it sometime.

Please sign in to comment.