Rémy F.

Results 99 comments of Rémy F.

How to quantify this slowdown ? is there any metric/tools that we could use ? (and also re-use once fixed, to show the performance increase)

I'm voting for the /dist/ folder removal because : - It makes the merging process impossible since all PR conflict with each other as soon as one od them changed...

- Github Action shall report status on PR open/update to help reviewer (+ add built artefact to the PR?) - Github Action shall build and publish artefact to NPM on...

> It'd be great to have advanced users test it before officially releasing it. @Connum those *advanced* users, who are capable of debugging OpenTypeJS, are probably also capable of running...

I made a [POC](https://github.com/yne/opentype.yne.js) on my fork that - build + lint + test on PR or merge - build + lint + test + [publish](https://github.com/yne/opentype.yne.js/actions/runs/4159570266/jobs/7195736158) on git tag [on...

Exactly, esbuild can output both `esm` and legacy (aka `global`) formats. But shall we move the default entrypoint to the ESM for the 2.0.0 ? using legacy `--global-name=opentype` format: ```html...

I can't find much popular hybrid file-processing oriented JS libraries. But here are some: - [mermaid](https://github.com/mermaid-js/mermaid) 53K :star: : platform-specific are separated in a [cli](https://github.com/mermaid-js/mermaid-cli) repo - [marked.js](https://github.com/markedjs/marked/blob/master/bin/marked.js) 29K :star:...

@ILOVEPIE you are right. I'll keep the original naming.

@Connum I dont understand. I see in they package.json that they pull opentype.js so they will still get the dist files ... did i missed something from they repo ?

You could even automate this build using a [github action](https://github.com/yne/opentype.yne.js/blob/master/.github/workflows/page.yml) that build opentype dist file and deploy the static website (with your built files)