autometrics-dev / autometrics-ts

Easily add metrics to your system – and actually understand them using automatically customized Prometheus queries

Home Page:https://autometrics.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

πŸ› Low priority: Remove tests from autometrics package distro

brettimus opened this issue Β· comments

Environment information

n/a

What happened?

When you npm i @auotmetrics/autometrics and inspect node_modules, it seems we include our tests folder.

% ls -1 node_modules/@autometrics/autometrics
README.md
dist
package.json
reference
src
tests
tsconfig.json
tsup.config.ts
typedoc.json

We can use an .npmignore to exclude these from the package. Just need to be careful, as using .npmignore means npm wil no longer respect .gitignore

Expected result

I wouldn't expect tests to be packaged with the autometrics library

I think this should be resolved with #131 too.