omnidan / node-emoji

💖 simple emoji support for node.js projects
MIT License
1.31k stars 241 forks source link

fix: pin skin-tone to ^2.0.0 and add renovate.json ignoreDeps #157

Closed JoshuaKGoldberg closed 11 months ago

JoshuaKGoldberg commented 11 months ago

PR Checklist

Overview

Continues #154 by pinning the last ESM-only dependency, skin-tone, to its pre-#142 version. See https://github.com/omnidan/node-emoji/issues/153#issuecomment-1819534057.

I don't know why the CJS test from #154 passed with skin-tone (exports oddity? race condition?) so manually disables the ESM-only dependencies in renove.json. Which is ... most of them.

codecov[bot] commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ac996a3) 100.00% compared to head (dcd6c43) 100.00%. Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #157 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 14 14 Lines 263 263 Branches 28 28 ========================================= Hits 263 263 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.