fix(typescript): rename index.js to index.ts to expose some components #14358
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #14165
Changelog
Changed
index.js
files toindex.ts
to make sure export statements innode_modules/@carbon/react/lib/index.d.ts
of a TS React project can be resolved by the TypeScript compiler. (Reference: How TypeScript resolves modules)(Note: This PR renames
index.js
files that only reference TS files. There are still many components that cannot be imported in a TS project. Fixing the problem for all components in a single PR may lead to a huge PR which would be hard to test and review, so I only fix the easiest ones in this PR.)Testing / Reviewing
DatePickerInput
by following the steps below.yarn build
npx create-react-app test-carbon-import-ts --template typescript
@carbon/react
:npm i @carbon/react
cp -r /root/my-fork/carbon/packages/react/lib /root/tmp/test-carbon-import-ts/node_modules/@carbon/react/
src/App.tsx
in the new project and verify those components are imported correctly.