{"action":"created","issue":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/8665","labels_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8665/labels{/name}","comments_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8665/comments","events_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8665/events","html_url":"https://github.com/cms-sw/cmssw/pull/8665","id":66372586,"number":8665,"title":"ZDC DQM update for 75X","user":{"login":"BetterWang","id":5550547,"avatar_url":"https://avatars.githubusercontent.com/u/5550547?v=3","gravatar_id":"","url":"https://api.github.com/users/BetterWang","html_url":"https://github.com/BetterWang","followers_url":"https://api.github.com/users/BetterWang/followers","following_url":"https://api.github.com/users/BetterWang/following{/other_user}","gists_url":"https://api.github.com/users/BetterWang/gists{/gist_id}","starred_url":"https://api.github.com/users/BetterWang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BetterWang/subscriptions","organizations_url":"https://api.github.com/users/BetterWang/orgs","repos_url":"https://api.github.com/users/BetterWang/repos","events_url":"https://api.github.com/users/BetterWang/events{/privacy}","received_events_url":"https://api.github.com/users/BetterWang/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/comparison-available","name":"comparison-available","color":"009800"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/dqm-pending","name":"dqm-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/orp-pending","name":"orp-pending","color":"fbca04"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/pending-signatures","name":"pending-signatures","color":"eb6420"},{"url":"https://api.github.com/repos/cms-sw/cmssw/labels/tests-approved","name":"tests-approved","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/cms-sw/cmssw/milestones/51","html_url":"https://github.com/cms-sw/cmssw/milestones/Next%20CMSSW_7_5_X","labels_url":"https://api.github.com/repos/cms-sw/cmssw/milestones/51/labels","id":982516,"number":51,"title":"Next CMSSW_7_5_X","description":"","creator":{"login":"nclopezo","id":1944922,"avatar_url":"https://avatars.githubusercontent.com/u/1944922?v=3","gravatar_id":"","url":"https://api.github.com/users/nclopezo","html_url":"https://github.com/nclopezo","followers_url":"https://api.github.com/users/nclopezo/followers","following_url":"https://api.github.com/users/nclopezo/following{/other_user}","gists_url":"https://api.github.com/users/nclopezo/gists{/gist_id}","starred_url":"https://api.github.com/users/nclopezo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nclopezo/subscriptions","organizations_url":"https://api.github.com/users/nclopezo/orgs","repos_url":"https://api.github.com/users/nclopezo/repos","events_url":"https://api.github.com/users/nclopezo/events{/privacy}","received_events_url":"https://api.github.com/users/nclopezo/received_events","type":"User","site_admin":false},"open_issues":45,"closed_issues":359,"state":"open","created_at":"2015-02-17T08:21:25Z","updated_at":"2015-04-09T15:07:55Z","due_on":"2015-03-02T23:00:00Z","closed_at":null},"comments":8,"created_at":"2015-04-04T22:13:47Z","updated_at":"2015-04-09T15:29:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/8665","html_url":"https://github.com/cms-sw/cmssw/pull/8665","diff_url":"https://github.com/cms-sw/cmssw/pull/8665.diff","patch_url":"https://github.com/cms-sw/cmssw/pull/8665.patch"},"body":"Ported ZDC DQM plots from 53X pPb run to 75X."},"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/issues/comments/91265093","html_url":"https://github.com/cms-sw/cmssw/pull/8665#issuecomment-91265093","issue_url":"https://api.github.com/repos/cms-sw/cmssw/issues/8665","id":91265093,"user":{"login":"deguio","id":3207679,"avatar_url":"https://avatars.githubusercontent.com/u/3207679?v=3","gravatar_id":"","url":"https://api.github.com/users/deguio","html_url":"https://github.com/deguio","followers_url":"https://api.github.com/users/deguio/followers","following_url":"https://api.github.com/users/deguio/following{/other_user}","gists_url":"https://api.github.com/users/deguio/gists{/gist_id}","starred_url":"https://api.github.com/users/deguio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deguio/subscriptions","organizations_url":"https://api.github.com/users/deguio/orgs","repos_url":"https://api.github.com/users/deguio/repos","events_url":"https://api.github.com/users/deguio/events{/privacy}","received_events_url":"https://api.github.com/users/deguio/received_events","type":"User","site_admin":false},"created_at":"2015-04-09T15:29:44Z","updated_at":"2015-04-09T15:29:44Z","body":"ciao @BetterWang \r\nmy comments are not about the content:\r\n- could you remove the commented lines where they are not needed? this is the case for example for the newly added ZDCMonitorClient.h ZDCMonitorClient.cc\r\n- not sure which spacing convention you have chosen and why. it is difficult to understand what the content of this PR is since possible changes are hidden in reformatting changes. it would be nice to have this decoupled. the long list of commits doesn't help either.\r\n\r\nF."}}