fixes #951 #963

Merged
merged 4 commits into from Apr 9, 2013

Projects

None yet

2 participants

@guyzmo
Contributor
guyzmo commented Mar 25, 2013

fixes #951:

  • added es3 config option
  • made es5 the default
  • added info warning if es5 is set
  • added function to encapsulate version state
@guyzmo
Contributor
guyzmo commented Mar 25, 2013

based on PR #960 (cf ticket #950).
All tests pass, but as I had to modify most of them, I think it still needs thorough testing.

@guyzmo
Contributor
guyzmo commented Mar 27, 2013

PR rebased. tests passed.

guyzmo added some commits Mar 22, 2013
@guyzmo guyzmo added A LOT of tests
renamed isMozOrESNext to isAfterESNext;
added isAfterES5;
refactored ESnext/Moz support to raise errors on future features;
fixed several parsing issues

Signed-off-by: Bernard `Guyzmo` Pratz <guyzmo+github@m0g.net>
e9e8186
@guyzmo guyzmo updated automatic comma insertion support (cf #950) 2980237
@guyzmo guyzmo added line breaking error on 'return' statement with and without asi …
…option (when it's broken, fix it).
38d14b4
@guyzmo guyzmo added es3 config option and updated tests to match es3 ;
made es5 the default and updated tests to remove es5 ;
added info message to tell about es5 being default ;
added functions to encapsulate management of current version state

Signed-off-by: Bernard `Guyzmo` Pratz <guyzmo+github@m0g.net>
c001e7c
@valueof
Member
valueof commented Mar 30, 2013

Is this patch still relevant?

@valueof valueof merged commit c001e7c into jshint:master Apr 9, 2013

1 check passed

default The Travis build passed
Details
@guyzmo guyzmo deleted the guyzmo:951 branch Apr 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment