These are chat archives for opal/opal

14th
Nov 2015
Michał Kalbarczyk
@fazibear
Nov 14 2015 10:20
@meh travis log:
$ bundle exec rake $RUN
mkdir -p tmp
ruby -rbundler/setup bin/opal -ce '23' > tmp/jshint.js
jshint --verbose tmp/jshint.js
tmp/jshint.js: line 1757, col 7, 'BasicObjectClass' was used before it was defined. (W003)
tmp/jshint.js: line 1760, col 7, 'ObjectClass' was used before it was defined. (W003)
tmp/jshint.js: line 1763, col 7, 'ModuleClass' was used before it was defined. (W003)
tmp/jshint.js: line 1766, col 7, 'ClassClass' was used before it was defined. (W003)
tmp/jshint.js: line 1831, col 7, 'nil' was used before it was defined. (W003)
5 errors
rake aborted!
Command failed with status (2): [jshint --verbose tmp/jshint.js...]
/home/travis/build/opal/opal/tasks/testing.rake:166:in `block in <top (required)>'
/home/travis/.rvm/gems/ruby-2.2.0/bin/ruby_executable_hooks:15:in `eval'
/home/travis/.rvm/gems/ruby-2.2.0/bin/ruby_executable_hooks:15:in `<main>'
Tasks: TOP => jshint
(See full trace by running task with --trace)
The command "bundle exec rake $RUN" exited with 1.
Michał Kalbarczyk
@fazibear
Nov 14 2015 10:40
And its just fine on local machine:
✓ ~/dev/opal 🌴 fix-js-new ➜ bundle exec rake jshint SUITE=core                                                                                                                                                   
mkdir -p tmp
ruby -rbundler/setup bin/opal -ce '23' > tmp/jshint.js
jshint --verbose tmp/jshint.js
✓ ~/dev/opal 🌴 fix-js-new ➜
Vais Salikhov
@vais
Nov 14 2015 13:43
@fazibear Travis is using the latest version of jshint automatically. If you update jshint on your local machine, you will see the same problem. I already fixed it in this PR opal/opal#1189 just waiting for @meh or @elia to merge :wink:
@fazibear to be specific, this commit vais/opal@71c57c0 is what fixes the jshint problem.