JoshuaKGoldberg/create-typescript-app

๐Ÿš€ Feature: Test files shouldn't be published

JoshuaKGoldberg opened this issue ยท 5 comments

Bug Report Checklist

Overview

Right now, CTA repos publish all files under lib/:

"files": [
"bin/index.js",
"lib/",
"package.json",
"LICENSE.md",
"README.md"
],

...including *.test.* files. ๐Ÿ˜ฌ. That's a lot of extra file size on published packages for no real gain. I think *.test.* files should be excluded from the published outputs.

Additional Info

I'd previously fixed this in 7cdb836, but at some point switched away from .npmignore files and lost that.

๐Ÿ’–

Is this as simple as re-introducing an .npmignore with this content?:

*.test.*

I think so!

Let me PR - just heading out so hopefully tomorrow

๐ŸŽ‰ This is included in version v1.71.4 ๐ŸŽ‰

The release is available on:

Cheers! ๐Ÿ“ฆ๐Ÿš€

๐ŸŽ‰ This is included in version v1.72.0 ๐ŸŽ‰

The release is available on:

Cheers! ๐Ÿ“ฆ๐Ÿš€