webfont icon indicating copy to clipboard operation
webfont copied to clipboard

Exclude test files from rollup

Open juliangilbey opened this issue 3 years ago • 1 comments

Summary

Proposed changes

This PR brings the following changes:

  • Only rollup the run-time files; exclude test files. (Otherwise many errors are produced.)

How to test

Describe the tests that you ran to verify your changes:

  1. Run the build
  2. Observe that you don't get dozens of warnings about unrecognised functions such as "expect".

Test configuration

  • Node.js version (if applicable): 18.12.1

Checklist

  • [ ] I have added corresponding labels to this PR (like bug, enhancement...); it's a bug, but I don't know how to add labels
  • [x] My commits follow the Conventional Commits 1.0 Guidelines;
  • [x] My code follows the style guidelines of this project;
  • [x] I have performed a self-review of my own code;
  • [ ] I have mapped technical debts found on my changes; N/A
  • [ ] I have made changes to the documentation (if applicable); N/A
  • [x] My changes generate no new warnings or errors;

juliangilbey avatar Dec 23 '22 09:12 juliangilbey

I should have added; this patch was written by Israel Galadima

juliangilbey avatar Dec 23 '22 09:12 juliangilbey