Merge pull request #1288 from DaanHoogland/trailing-commas trailing commas in javascripts removedAccording to SonarQube this is a bug on internet explorer. It is the only 'blocker' level issue in cloudstack. @abhinandanprateek (@agneya2001) @bhaisaab @miguelaferreira should we enforce this or, as alternative, have it disabled in SonarQube?
* pr/1288: trailing commas in javascripts removed Signed-off-by: Rohit Yadav <rohit.ya...@shapeblue.com> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/5ab82d10 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/5ab82d10 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/5ab82d10 Branch: refs/heads/master Commit: 5ab82d10d99e7bca9147ac54348c3bc34889c946 Parents: b98a989 28ca933 Author: Rohit Yadav <rohit.ya...@shapeblue.com> Authored: Wed Jan 27 16:28:28 2016 +0100 Committer: Rohit Yadav <rohit.ya...@shapeblue.com> Committed: Wed Jan 27 16:28:29 2016 +0100 ---------------------------------------------------------------------- ui/plugins/plugins.js | 4 ++-- ui/plugins/quota/quota.js | 47 +++++++++++++++++++++--------------------- ui/scripts/accounts.js | 4 ++-- ui/scripts/cloudStack.js | 2 +- ui/scripts/instances.js | 2 +- ui/scripts/metrics.js | 20 +++++++++--------- ui/scripts/system.js | 4 ++-- 7 files changed, 41 insertions(+), 42 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/5ab82d10/ui/scripts/cloudStack.js ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/5ab82d10/ui/scripts/metrics.js ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/5ab82d10/ui/scripts/system.js ----------------------------------------------------------------------
