google / google/gts

Add /test to .npmignore

Open
#544 1 comment 0 reactions 0 assignees View on GitHub
type: feature request
Dominant language
TypeScript
Stars
5.3k
Forks
212
PR merge metrics
No merged PRs in 30d

Description

In a new GTS setup when one drops a `.spec.ts` file into `/test` and `import`s anything from `mocha` or `chai`, this leads to an ESLint warning for [no-unpublished-import](https://github.com/mysticatea/eslint-plugin-node/blob/v11.1.0/docs/rules/no-unpublished-import.md).

As the 💡 in *Rule Details* suggests, a `.npmignore` holding one line for `/test` could lead to a less disruptive experience for new setups.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.