Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

fix: include /test/ folder in tsconfig.json #89

Closed
wants to merge 1 commit into from

Conversation

erickzhao
Copy link
Member

Our various test folders are called /test/, but #80 called them /tests/ instead.

@erickzhao erickzhao requested a review from ckerr July 2, 2021 23:49
@erickzhao
Copy link
Member Author

Build is failing.

So the build seems to be passing with this commit reverted, and if include is just "include": ["src/**/*"].

However, when the build passes, I get these lint errors when looking at a test file:

Parsing error: "parserOptions.project" has been set for @typescript-eslint/parser.
The file does not match your project config: modules/bot/test/github-client.spec.ts.
The file must be included in at least one of the projects provided

@erickzhao
Copy link
Member Author

Since this PR doesn't solve the core issue, I'm going to close it and track the problem in #92.

@erickzhao erickzhao closed this Jul 3, 2021
@erickzhao erickzhao deleted the fix/test-tsconfig branch July 3, 2021 03:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants