You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
json2.js is only needed for IE < 8 according to http://caniuse.com/#feat=json – can we stop shipping this and leave it to users to work out whether they need legacy IE support? I would vote to remove it and leave a helpful note about legacy support in the documentation.
This is a more complex one. The version of Underscore that Judge provides is very out of date and, in retrospect, I don't like the praxis of providing dependencies as part of this library. When Judge was first written, JS dependency management tools like Bower and npm either didn't exist, or had not been widely adopted. Nowadays it's still a pretty fragmented state of affairs, but I think users can be expected to take care of it for themselves. I think the options are:
Remove Underscore from the Judge project. Expect users to install it into their applications
Remove Underscore from the Judge project completely, including refactoring it out of judge.js
Continue to ship Underscore with Judge, make more of an effort to keep it up to date
Thoughts?
The text was updated successfully, but these errors were encountered:
I'd vote for (iii). If it's a true dependency I believe it should be bundled in the gem for ease of implemenation, but I agree we should keep it up to date.
The other part of the discussion here would be what version would these changes would go into? These are potenially non-backwards compatible since we'd officially be dropping some browser support. This hints at bumping the version to v3, but that's also a big jump follow the version history of the gem.
Opening this discussion.
json2.js
is only needed for IE < 8 according to http://caniuse.com/#feat=json – can we stop shipping this and leave it to users to work out whether they need legacy IE support? I would vote to remove it and leave a helpful note about legacy support in the documentation.judge.js
Thoughts?
The text was updated successfully, but these errors were encountered: