kentcdodds / mdx-bundler

🦤 Give me MDX/TSX strings and I'll give you back a component you can render. Supports imports!
MIT License
1.78k stars 75 forks source link

fix: missing type definition #176

Closed vladmoroz closed 1 year ago

vladmoroz commented 2 years ago

What: Fixes MDXContentProps type missing from types.d.ts

Why: This issue prevented a basic Next.js setup with mdx-bundler from building. Fixes #75.

How: Add MDXContentProps to types.d.ts and import it in ./client

Checklist:

kentcdodds commented 1 year ago

Unfortunately there's an issue with the types. Do you mind looking into that?

codecov[bot] commented 1 year ago

Codecov Report

Merging #176 (94f32a4) into main (b96327e) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #176   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          296       292    -4     
  Branches         1         1           
=========================================
- Hits           296       292    -4     
Impacted Files Coverage Δ
src/client.js 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

kentcdodds commented 1 year ago

@All-contributors please add @vladmoroz for code

allcontributors[bot] commented 1 year ago

@kentcdodds

I've put up a pull request to add @vladmoroz! :tada:

github-actions[bot] commented 1 year ago

:tada: This PR is included in version 9.2.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: