GitHub PullRequestEvent at 2014-03-12 15:42:21
Repository: non-stupid-digest-assets
Actor: kronn
Payload: {"action":"opened","number":12,"pull_request":{"url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/12","id":13476779,"html_url":"https://github.com/alexspeller/non-stupid-digest-assets/pull/12","diff_url":"https://github.com/alexspeller/non-stupid-digest-assets/pull/12.diff","patch_url":"https://github.com/alexspeller/non-stupid-digest-assets/pull/12.patch","issue_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/issues/12","number":12,"state":"open","title":"Preserve mtime when creating the non-digested file","user":{"login":"kronn","id":72198,"avatar_url":"https://gravatar.com/avatar/b474b21a15938494d690805e9b319a12?d=https%3A%2F%2Fidenticons.github.com%2F6a158617d515b6fcd22a06e8b952d135.png&r=x","gravatar_id":"b474b21a15938494d690805e9b319a12","url":"https://api.github.com/users/kronn","html_url":"https://github.com/kronn","followers_url":"https://api.github.com/users/kronn/followers","following_url":"https://api.github.com/users/kronn/following{/other_user}","gists_url":"https://api.github.com/users/kronn/gists{/gist_id}","starred_url":"https://api.github.com/users/kronn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kronn/subscriptions","organizations_url":"https://api.github.com/users/kronn/orgs","repos_url":"https://api.github.com/users/kronn/repos","events_url":"https://api.github.com/users/kronn/events{/privacy}","received_events_url":"https://api.github.com/users/kronn/received_events","type":"User","site_admin":false},"body":"FileUtils.cp does not copy the file-attributes from the source to the new\r\nfile.  With copy_file, this is possible. The third argument needs to be\r\n\"trueish\", so a descriptive symbol suffices.\r\n\r\nIn effect, this copies over the mtime from the assets. This helps Webservers\r\nto generate ETags based on the mtime which in turn can prevent the need to\r\nretransfer a file if it hasn't changed since the last request.\r\n\r\nEssentially, this patch is saving the money of all those who are paying for\r\nbandwidth. As a side-effect, I'm not confused when I see the directory\r\nlisting.","created_at":"2014-03-12T15:42:21Z","updated_at":"2014-03-12T15:42:21Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/12/commits","review_comments_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/12/comments","review_comment_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/comments/{number}","comments_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/issues/12/comments","statuses_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/statuses/c9bc1cac9cc0be3ef38bd3349ce02991360ed615","head":{"label":"kronn:patch-1","ref":"patch-1","sha":"c9bc1cac9cc0be3ef38bd3349ce02991360ed615","user":{"login":"kronn","id":72198,"avatar_url":"https://gravatar.com/avatar/b474b21a15938494d690805e9b319a12?d=https%3A%2F%2Fidenticons.github.com%2F6a158617d515b6fcd22a06e8b952d135.png&r=x","gravatar_id":"b474b21a15938494d690805e9b319a12","url":"https://api.github.com/users/kronn","html_url":"https://github.com/kronn","followers_url":"https://api.github.com/users/kronn/followers","following_url":"https://api.github.com/users/kronn/following{/other_user}","gists_url":"https://api.github.com/users/kronn/gists{/gist_id}","starred_url":"https://api.github.com/users/kronn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kronn/subscriptions","organizations_url":"https://api.github.com/users/kronn/orgs","repos_url":"https://api.github.com/users/kronn/repos","events_url":"https://api.github.com/users/kronn/events{/privacy}","received_events_url":"https://api.github.com/users/kronn/received_events","type":"User","site_admin":false},"repo":{"id":17671712,"name":"non-stupid-digest-assets","full_name":"kronn/non-stupid-digest-assets","owner":{"login":"kronn","id":72198,"avatar_url":"https://gravatar.com/avatar/b474b21a15938494d690805e9b319a12?d=https%3A%2F%2Fidenticons.github.com%2F6a158617d515b6fcd22a06e8b952d135.png&r=x","gravatar_id":"b474b21a15938494d690805e9b319a12","url":"https://api.github.com/users/kronn","html_url":"https://github.com/kronn","followers_url":"https://api.github.com/users/kronn/followers","following_url":"https://api.github.com/users/kronn/following{/other_user}","gists_url":"https://api.github.com/users/kronn/gists{/gist_id}","starred_url":"https://api.github.com/users/kronn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kronn/subscriptions","organizations_url":"https://api.github.com/users/kronn/orgs","repos_url":"https://api.github.com/users/kronn/repos","events_url":"https://api.github.com/users/kronn/events{/privacy}","received_events_url":"https://api.github.com/users/kronn/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/kronn/non-stupid-digest-assets","description":"Fix the Rails 4 asset pipeline to generate non-digest along with digest assets","fork":true,"url":"https://api.github.com/repos/kronn/non-stupid-digest-assets","forks_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/forks","keys_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/keys{/key_id}","collaborators_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/teams","hooks_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/hooks","issue_events_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/issues/events{/number}","events_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/events","assignees_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/assignees{/user}","branches_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/branches{/branch}","tags_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/tags","blobs_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/git/refs{/sha}","trees_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/git/trees{/sha}","statuses_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/statuses/{sha}","languages_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/languages","stargazers_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/stargazers","contributors_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/contributors","subscribers_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/subscribers","subscription_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/subscription","commits_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/commits{/sha}","git_commits_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/git/commits{/sha}","comments_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/comments{/number}","issue_comment_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/issues/comments/{number}","contents_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/contents/{+path}","compare_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/compare/{base}...{head}","merges_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/merges","archive_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/downloads","issues_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/issues{/number}","pulls_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/pulls{/number}","milestones_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/milestones{/number}","notifications_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/labels{/name}","releases_url":"https://api.github.com/repos/kronn/non-stupid-digest-assets/releases{/id}","created_at":"2014-03-12T15:23:47Z","updated_at":"2014-03-12T15:42:21Z","pushed_at":"2014-03-12T15:41:48Z","git_url":"git://github.com/kronn/non-stupid-digest-assets.git","ssh_url":"git@github.com:kronn/non-stupid-digest-assets.git","clone_url":"https://github.com/kronn/non-stupid-digest-assets.git","svn_url":"https://github.com/kronn/non-stupid-digest-assets","homepage":null,"size":254,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":false,"has_downloads":true,"has_wiki":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"alexspeller:master","ref":"master","sha":"a15c9bbc3c02827f2c63514fc466e3a95f88b464","user":{"login":"alexspeller","id":1217,"avatar_url":"https://gravatar.com/avatar/318324ac607560170ad7d0e61ce0288b?d=https%3A%2F%2Fidenticons.github.com%2F6a61d423d02a1c56250dc23ae7ff12f3.png&r=x","gravatar_id":"318324ac607560170ad7d0e61ce0288b","url":"https://api.github.com/users/alexspeller","html_url":"https://github.com/alexspeller","followers_url":"https://api.github.com/users/alexspeller/followers","following_url":"https://api.github.com/users/alexspeller/following{/other_user}","gists_url":"https://api.github.com/users/alexspeller/gists{/gist_id}","starred_url":"https://api.github.com/users/alexspeller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexspeller/subscriptions","organizations_url":"https://api.github.com/users/alexspeller/orgs","repos_url":"https://api.github.com/users/alexspeller/repos","events_url":"https://api.github.com/users/alexspeller/events{/privacy}","received_events_url":"https://api.github.com/users/alexspeller/received_events","type":"User","site_admin":false},"repo":{"id":12982177,"name":"non-stupid-digest-assets","full_name":"alexspeller/non-stupid-digest-assets","owner":{"login":"alexspeller","id":1217,"avatar_url":"https://gravatar.com/avatar/318324ac607560170ad7d0e61ce0288b?d=https%3A%2F%2Fidenticons.github.com%2F6a61d423d02a1c56250dc23ae7ff12f3.png&r=x","gravatar_id":"318324ac607560170ad7d0e61ce0288b","url":"https://api.github.com/users/alexspeller","html_url":"https://github.com/alexspeller","followers_url":"https://api.github.com/users/alexspeller/followers","following_url":"https://api.github.com/users/alexspeller/following{/other_user}","gists_url":"https://api.github.com/users/alexspeller/gists{/gist_id}","starred_url":"https://api.github.com/users/alexspeller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alexspeller/subscriptions","organizations_url":"https://api.github.com/users/alexspeller/orgs","repos_url":"https://api.github.com/users/alexspeller/repos","events_url":"https://api.github.com/users/alexspeller/events{/privacy}","received_events_url":"https://api.github.com/users/alexspeller/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/alexspeller/non-stupid-digest-assets","description":"Fix the Rails 4 asset pipeline to generate non-digest along with digest assets","fork":false,"url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets","forks_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/forks","keys_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/keys{/key_id}","collaborators_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/teams","hooks_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/hooks","issue_events_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/issues/events{/number}","events_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/events","assignees_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/assignees{/user}","branches_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/branches{/branch}","tags_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/tags","blobs_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/git/refs{/sha}","trees_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/git/trees{/sha}","statuses_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/statuses/{sha}","languages_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/languages","stargazers_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/stargazers","contributors_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/contributors","subscribers_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/subscribers","subscription_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/subscription","commits_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/commits{/sha}","git_commits_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/git/commits{/sha}","comments_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/comments{/number}","issue_comment_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/issues/comments/{number}","contents_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/contents/{+path}","compare_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/compare/{base}...{head}","merges_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/merges","archive_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/downloads","issues_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/issues{/number}","pulls_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls{/number}","milestones_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/milestones{/number}","notifications_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/labels{/name}","releases_url":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/releases{/id}","created_at":"2013-09-20T19:35:26Z","updated_at":"2014-02-12T20:52:06Z","pushed_at":"2014-02-12T20:52:05Z","git_url":"git://github.com/alexspeller/non-stupid-digest-assets.git","ssh_url":"git@github.com:alexspeller/non-stupid-digest-assets.git","clone_url":"https://github.com/alexspeller/non-stupid-digest-assets.git","svn_url":"https://github.com/alexspeller/non-stupid-digest-assets","homepage":null,"size":254,"stargazers_count":60,"watchers_count":60,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"forks_count":14,"mirror_url":null,"open_issues_count":2,"forks":14,"open_issues":2,"watchers":60,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/12"},"html":{"href":"https://github.com/alexspeller/non-stupid-digest-assets/pull/12"},"issue":{"href":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/issues/12"},"comments":{"href":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/issues/12/comments"},"review_comments":{"href":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/12/comments"},"review_comment":{"href":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/pulls/12/commits"},"statuses":{"href":"https://api.github.com/repos/alexspeller/non-stupid-digest-assets/statuses/c9bc1cac9cc0be3ef38bd3349ce02991360ed615"}},"merged":false,"mergeable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"commits":1,"additions":2,"deletions":2,"changed_files":1}}