Skip to content

Commit

Permalink
zuul: drop IE6-7 tests
Browse files Browse the repository at this point in the history
IE6 and IE7 tests stopped working at some point, presumably because
some dependency of zuul or browserify added a dangling comma or some
feature that these browsers don't support.

I don't have the time/patience to debug this now. buffer probably works
in IE6-7 still, but I'm dropping the tests for it. If someone wants to
fix the issue and re-enable these tests, you're welcome to :)
  • Loading branch information
feross committed Mar 10, 2015
1 parent 2368cec commit 62cf522
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .zuul.yml
Expand Up @@ -9,7 +9,7 @@ browsers:
- name: safari
version: 5..latest
- name: ie
version: 6..latest
version: 8..latest
- name: opera
version: 11..latest
- name: iphone
Expand Down
4 changes: 3 additions & 1 deletion bin/download-node-tests.js
Expand Up @@ -56,7 +56,9 @@ function testfixer (filename) {

if (firstline) {
// require buffer explicitly
line = 'var Buffer = require(\'../../\').Buffer\nif (process.env.OBJECT_IMPL) Buffer.TYPED_ARRAY_SUPPORT = false\n' + line
line = 'var Buffer = require(\'../../\').Buffer\n' +
'if (process.env.OBJECT_IMPL) Buffer.TYPED_ARRAY_SUPPORT = false\n' +
line
firstline = false
}

Expand Down

0 comments on commit 62cf522

Please sign in to comment.