{"action":"created","issue":{"url":"https://api.github.com/repos/gionkunz/chartist-js/issues/132","labels_url":"https://api.github.com/repos/gionkunz/chartist-js/issues/132/labels{/name}","comments_url":"https://api.github.com/repos/gionkunz/chartist-js/issues/132/comments","events_url":"https://api.github.com/repos/gionkunz/chartist-js/issues/132/events","html_url":"https://github.com/gionkunz/chartist-js/issues/132","id":53752067,"number":132,"title":"yuicompressor-maven-plugin","user":{"login":"nKognito","id":612207,"avatar_url":"https://avatars.githubusercontent.com/u/612207?v=3","gravatar_id":"","url":"https://api.github.com/users/nKognito","html_url":"https://github.com/nKognito","followers_url":"https://api.github.com/users/nKognito/followers","following_url":"https://api.github.com/users/nKognito/following{/other_user}","gists_url":"https://api.github.com/users/nKognito/gists{/gist_id}","starred_url":"https://api.github.com/users/nKognito/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nKognito/subscriptions","organizations_url":"https://api.github.com/users/nKognito/orgs","repos_url":"https://api.github.com/users/nKognito/repos","events_url":"https://api.github.com/users/nKognito/events{/privacy}","received_events_url":"https://api.github.com/users/nKognito/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-08T13:50:02Z","updated_at":"2015-01-08T14:33:59Z","closed_at":null,"body":"I try to minify the source code with that plugin and it always fails.. Even if I use the minified version and try to minify my own js file (which uses chartist methods) it fails...\r\nHere are logs:\r\n\r\n```...chartist.js:line 926:column 19:missing name after . operator\r\n\t      constr.super = superProto;```\r\n```[ERROR] ...chartist.js:line 2111:column 26:missing name after . operator\r\n\t      Chartist.Line.super.constructor.call(this,```\r\n```[ERROR] ...chartist.js:line 2112:column 14:syntax error\r\n\t        query,```\r\n```[ERROR] ...chartist.js:line 2113:column 13:syntax error\r\n\t        data,```\r\n```[ERROR] ...chartist.js:line 2114:column 18:syntax error\r\n\t        Chartist.extend({}, defaultOptions, options),```\r\n```[ERROR] ...chartist.js:line 2115:column 26:syntax error\r\n\t        responsiveOptions);```\r\n```[ERROR] ...chartist.js:line 2357:column 25:missing name after . operator\r\n\t      Chartist.Bar.super.constructor.call(this,```\r\n```[ERROR] ...chartist.js:line 2358:column 14:syntax error\r\n\t        query,```\r\n```[ERROR] ...chartist.js:line 2359:column 13:syntax error\r\n\t        data,```\r\n```[ERROR] ...chartist.js:line 2360:column 18:syntax error\r\n\t        Chartist.extend({}, defaultOptions, options),```\r\n```[ERROR] ...chartist.js:line 2361:column 26:syntax error\r\n\t        responsiveOptions);```\r\n```[ERROR] ...chartist.js:line 2656:column 25:missing name after . operator\r\n\t      Chartist.Pie.super.constructor.call(this,```\r\n```[ERROR] ...chartist.js:line 2657:column 14:syntax error\r\n\t        query,```\r\n```[ERROR] ...chartist.js:line 2658:column 13:syntax error\r\n\t        data,```\r\n```[ERROR] ...chartist.js:line 2659:column 18:syntax error\r\n\t        Chartist.extend({}, defaultOptions, options),```\r\n```[ERROR] ...chartist.js:line 2660:column 26:syntax error\r\n\t        responsiveOptions);```\r\n```[ERROR] ...chartist.js:line 1:column 0:Compilation produced 16 syntax errors.```"},"comment":{"url":"https://api.github.com/repos/gionkunz/chartist-js/issues/comments/69186421","html_url":"https://github.com/gionkunz/chartist-js/issues/132#issuecomment-69186421","issue_url":"https://api.github.com/repos/gionkunz/chartist-js/issues/132","id":69186421,"user":{"login":"gionkunz","id":2203704,"avatar_url":"https://avatars.githubusercontent.com/u/2203704?v=3","gravatar_id":"","url":"https://api.github.com/users/gionkunz","html_url":"https://github.com/gionkunz","followers_url":"https://api.github.com/users/gionkunz/followers","following_url":"https://api.github.com/users/gionkunz/following{/other_user}","gists_url":"https://api.github.com/users/gionkunz/gists{/gist_id}","starred_url":"https://api.github.com/users/gionkunz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gionkunz/subscriptions","organizations_url":"https://api.github.com/users/gionkunz/orgs","repos_url":"https://api.github.com/users/gionkunz/repos","events_url":"https://api.github.com/users/gionkunz/events{/privacy}","received_events_url":"https://api.github.com/users/gionkunz/received_events","type":"User","site_admin":false},"created_at":"2015-01-08T14:33:59Z","updated_at":"2015-01-08T14:33:59Z","body":"You're right. Chartist is for browsers that support ECMAScript 5+ and even though `super` is a reserved words / keyword, the specification states clearly for object accessors ( `MemberExpression . IdentifierName`) an `IdentifierName` can be used and not an `Identifier`. An `IdentifierName` is almost any Unicode 3 expression and doesn't care about reserved words. Reserved words are only forbidden to use in `Identifiers` which as per the specification is an `IdentifierName` **but not** `ReservedWord`. \r\n\r\nAnyway, I'd say the issue is with YUI Compressor not supporting ECMA Script 5.1 properly and should be fixed there. I know they had a few GitHub issues around reserved words like `private` and `class` but they always just reply with that those words are reserved, which if you ask me is not correct. They should fix this.\r\n\r\nAlso I'd not recommend using YUI Compressor as they moved away from using it themselves ( http://www.yuiblog.com/blog/2012/10/16/state-of-yui-compressor/ ) years ago :smile: \r\n\r\nI personally prefer UglifyJS or even Google Closure Compiler over YUICompressor (I was using YUI Compressor with some Ruby Projects) and I'd recommend you to switch to uglifyjs-maven-plugin.\r\n\r\nCheers\r\nGion"}}