Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add modular directory to targetLocations checked by typecheck #2470

Merged
merged 8 commits into from
Oct 12, 2023

Conversation

ImmanuelBaskaran
Copy link
Contributor

This fixes an issue where the modular folder is ignored when running the typecheck command.

The modular folder in project contains the setupTests.ts file which is used to import packages such as 'testing-library/jest-dom' which extend the types of jest's matchers.

Currently this directory is ignored which causes typecheck to fail if a global import is used in the setupTests.ts

@changeset-bot
Copy link

changeset-bot bot commented Oct 12, 2023

🦋 Changeset detected

Latest commit: cdb1a2f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
modular-scripts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coveralls
Copy link
Collaborator

coveralls commented Oct 12, 2023

Coverage Status

coverage: 44.366%. remained the same when pulling 7462d53 on ImmanuelBaskaran:main into 71e3f5d on jpmorganchase:main.

yarn.lock Outdated Show resolved Hide resolved
@AlbertoBrusa AlbertoBrusa merged commit a4fb17a into jpmorganchase:main Oct 12, 2023
20 checks passed
@github-actions github-actions bot mentioned this pull request Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants