{"action":"created","issue":{"url":"https://api.github.com/repos/sindresorhus/gulp-rev/issues/83","labels_url":"https://api.github.com/repos/sindresorhus/gulp-rev/issues/83/labels{/name}","comments_url":"https://api.github.com/repos/sindresorhus/gulp-rev/issues/83/comments","events_url":"https://api.github.com/repos/sindresorhus/gulp-rev/issues/83/events","html_url":"https://github.com/sindresorhus/gulp-rev/issues/83","id":54835443,"number":83,"title":"Merge is broken on 3.0.0","user":{"login":"montogeek","id":1002461,"avatar_url":"https://avatars.githubusercontent.com/u/1002461?v=3","gravatar_id":"","url":"https://api.github.com/users/montogeek","html_url":"https://github.com/montogeek","followers_url":"https://api.github.com/users/montogeek/followers","following_url":"https://api.github.com/users/montogeek/following{/other_user}","gists_url":"https://api.github.com/users/montogeek/gists{/gist_id}","starred_url":"https://api.github.com/users/montogeek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/montogeek/subscriptions","organizations_url":"https://api.github.com/users/montogeek/orgs","repos_url":"https://api.github.com/users/montogeek/repos","events_url":"https://api.github.com/users/montogeek/events{/privacy}","received_events_url":"https://api.github.com/users/montogeek/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-01-20T02:10:56Z","updated_at":"2015-10-27T18:28:11Z","closed_at":null,"body":"I just updated the ```appendExisting``` (https://github.com/sindresorhus/gulp-rev/issues/65#issuecomment-68610591) option to ```merge``` (https://github.com/sindresorhus/gulp-rev/releases/tag/v3.0.0) and now the manifest file is not being updated :(\r\n\r\nMy gulpfile.js\r\n\r\n```\r\n\r\nvar publicPath = 'public/'\r\ngulp.task('js', function() {\r\n  return gulp.src(scriptsPath + 'main.js', { base: path.join(process.cwd(), 'public/app') } )\r\n    .pipe(browserify({\r\n      insertGlobals: true,\r\n      debug: true\r\n    }))\r\n    // Bundle to a single file\r\n    .pipe(concat('scripts/app.js'))\r\n    .pipe(uglify())\r\n    .pipe(rev())\r\n    .pipe(gulp.dest(publicPath))\r\n    .pipe(rev.manifest({\r\n      base: publicPath,\r\n      merge: true\r\n    }))\r\n    .pipe(gulp.dest(publicPath))\r\n});\r\n```\r\n\r\nThe existing rev-manifest.json file is \r\n```\r\n{\r\n  \"styles/main.css\": \"styles/main-23ea264f.min.css\"\r\n}\r\n```\r\n\r\nThe app.js is correctly minified and rev'd but the manifest file is not updated."},"comment":{"url":"https://api.github.com/repos/sindresorhus/gulp-rev/issues/comments/151601685","html_url":"https://github.com/sindresorhus/gulp-rev/issues/83#issuecomment-151601685","issue_url":"https://api.github.com/repos/sindresorhus/gulp-rev/issues/83","id":151601685,"user":{"login":"joezimjs","id":1313898,"avatar_url":"https://avatars.githubusercontent.com/u/1313898?v=3","gravatar_id":"","url":"https://api.github.com/users/joezimjs","html_url":"https://github.com/joezimjs","followers_url":"https://api.github.com/users/joezimjs/followers","following_url":"https://api.github.com/users/joezimjs/following{/other_user}","gists_url":"https://api.github.com/users/joezimjs/gists{/gist_id}","starred_url":"https://api.github.com/users/joezimjs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joezimjs/subscriptions","organizations_url":"https://api.github.com/users/joezimjs/orgs","repos_url":"https://api.github.com/users/joezimjs/repos","events_url":"https://api.github.com/users/joezimjs/events{/privacy}","received_events_url":"https://api.github.com/users/joezimjs/received_events","type":"User","site_admin":false},"created_at":"2015-10-27T18:28:11Z","updated_at":"2015-10-27T18:28:11Z","body":"I think the merge option should just have us pass a path to the original manifest, or `rev.manifest` should just write the file itself instead of letting `gulp.dest` handle it."}}