image-sequencer icon indicating copy to clipboard operation
image-sequencer copied to clipboard

Use Modern JS

Open harshkhandeparkar opened this issue 3 years ago • 2 comments

Since we are thinking of dropping node 10 support in https://github.com/publiclab/image-sequencer/issues/1935 and because most browsers support ES6 and beyond, we can:

  • [ ] Remove all vars and use let/const instead.
  • [ ] Use async/await and promises.
  • [ ] Even use ES6 imports by converting normal JS files to .mjs. (which will be compiled by browserify into a single package)
  • [ ] Output ESM for NodeJS instead of commonjs.
  • [ ] Worker Threads (NodeJS Only)

ref: https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c

This can be done file by file without issues. Anything else?

harshkhandeparkar avatar Jul 31 '21 14:07 harshkhandeparkar

Hi! I would like to work on this.

Bl4ck-h00d avatar Aug 27 '21 14:08 Bl4ck-h00d

Please assign me this issue . I would really like to work on it .

saurabhmehta1601 avatar Dec 27 '22 07:12 saurabhmehta1601