-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor codebase to use ES6+ features #216
Labels
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The code syntax the tests are written in is pretty much outdated and in some parts quite hard to read.
To improve that I suggest to use newer JavaScript language and DOM features like classes,
let
andconst
, modules, destructuring, newArray
methods, etc.Having said that, this will also break the site for browsers that don't support those features.
So the features should be chosen carefully and when possible be polyfilled if not supported.
Furthermore, backwards compatibility shouldn't be a problem anymore once the data needed for #57 is in place. When that's available there is no need to run the tests on old browsesr anymore.
Sebastian
The text was updated successfully, but these errors were encountered: