)]}'
{"app/js/controllers/project.js":[{"author":{"_account_id":18158,"name":"Bruno Tavares","email":"btavare@thoughtworks.com","username":"bltavares"},"change_message_id":"b58738c92b5fe32961d976160453f1f5ae631a81","unresolved":false,"context_lines":[{"line_number":6,"context_line":" * @ngInject"},{"line_number":7,"context_line":" */"},{"line_number":8,"context_line":"function ProjectCtrl($http, projectName) {"},{"line_number":9,"context_line":"  "},{"line_number":10,"context_line":"  // ViewModel"},{"line_number":11,"context_line":"  var vm \u003d this;"},{"line_number":12,"context_line":""}],"source_content_type":"text/javascript","patch_set":1,"id":"9a1a9d01_a2118db1","line":9,"updated":"2015-10-02 15:21:22.000000000","message":"(nit) remove whitespace","commit_id":"3c552c7d707f7f28d26f83acbaed636e95127c6b"},{"author":{"_account_id":18158,"name":"Bruno Tavares","email":"btavare@thoughtworks.com","username":"bltavares"},"change_message_id":"b58738c92b5fe32961d976160453f1f5ae631a81","unresolved":false,"context_lines":[{"line_number":25,"context_line":"      var totalFail \u003d 0;"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"      timedata.job_data.forEach(function(job) {"},{"line_number":28,"context_line":"        if (!(job.job_name in jobs)) {"},{"line_number":29,"context_line":"          var job_metrics \u003d {"},{"line_number":30,"context_line":"            name: job.job_name,"},{"line_number":31,"context_line":"            passes: 0,"}],"source_content_type":"text/javascript","patch_set":1,"id":"9a1a9d01_c21cc179","line":28,"updated":"2015-10-02 15:21:22.000000000","message":"Using the `(x in y)` construct in javascript is not ideal, as it could access other properties defined on the prototype chain, and many other issues, eg: `.length`. [1]\n\nIf you want to check if the value is not defined on that object, use `!jobs[job.job_name]`\n\n[1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/in","commit_id":"3c552c7d707f7f28d26f83acbaed636e95127c6b"},{"author":{"_account_id":18158,"name":"Bruno Tavares","email":"btavare@thoughtworks.com","username":"bltavares"},"change_message_id":"b58738c92b5fe32961d976160453f1f5ae631a81","unresolved":false,"context_lines":[{"line_number":34,"context_line":"          };"},{"line_number":35,"context_line":"          jobs[job.job_name] \u003d job_metrics;"},{"line_number":36,"context_line":"        }"},{"line_number":37,"context_line":"        "},{"line_number":38,"context_line":"        totalPass +\u003d job.pass;"},{"line_number":39,"context_line":"        totalFail +\u003d job.fail;"},{"line_number":40,"context_line":""}],"source_content_type":"text/javascript","patch_set":1,"id":"9a1a9d01_e21f85b5","line":37,"updated":"2015-10-02 15:21:22.000000000","message":"(nit) remove whitespace","commit_id":"3c552c7d707f7f28d26f83acbaed636e95127c6b"}]}