ESM broken since version 1.2.1 #13

diegone · 2022-03-02T05:55:29Z

CJS require works but ESM import throws an exception from version 1.2.1 to 1.2.3. Version 1.2.0 works fine.

kefniark · 2022-03-02T07:28:56Z

Can you explain more about the error you get and how you are using it?

I just tested the last version with a simple HTML code and <script type="module"> and it work without any trouble with the ESM file.

<!DOCTYPE html>
<html>
  <head>
    <meta charset="UTF-8">
  </head>
  <body>
    <script type="module">
      import { detect } from './dist/tinyld.esm.js'

      console.log(detect('test my text'))
    </script>
  </body>
</html>

diegone · 2022-03-02T16:05:02Z

Sorry, I should have been more specific. This happened to me using Node.js 16.13.1. If I do import('tinyld') from the REPL or import tinild from 'tinild' from my module, I get a dump of the source and a SyntaxError: Unexpected token 'export' exception. The same works fine with 1.2.0.

kefniark · 2022-07-23T15:40:24Z

This should be solved in last version 👍

kefniark mentioned this issue Jul 23, 2022

Fix deno issue, Node ESM import and few issues #15

Merged

kefniark closed this as completed Jul 23, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ESM broken since version 1.2.1 #13

ESM broken since version 1.2.1 #13

diegone commented Mar 2, 2022

kefniark commented Mar 2, 2022 •

edited

Loading

diegone commented Mar 2, 2022 •

edited

Loading

kefniark commented Jul 23, 2022 •

edited

Loading

ESM broken since version 1.2.1 #13

ESM broken since version 1.2.1 #13

Comments

diegone commented Mar 2, 2022

kefniark commented Mar 2, 2022 • edited Loading

diegone commented Mar 2, 2022 • edited Loading

kefniark commented Jul 23, 2022 • edited Loading

kefniark commented Mar 2, 2022 •

edited

Loading

diegone commented Mar 2, 2022 •

edited

Loading

kefniark commented Jul 23, 2022 •

edited

Loading