Skip to content

Fix ref in types#668

Merged
wooorm merged 1 commit into
remarkjs:mainfrom
ChristianMurphy:types/remove-ref-from-instrinsic-reflection
Jan 2, 2022
Merged

Fix ref in types#668
wooorm merged 1 commit into
remarkjs:mainfrom
ChristianMurphy:types/remove-ref-from-instrinsic-reflection

Conversation

@ChristianMurphy

Copy link
Copy Markdown
Member

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

Follow up to #666 (comment) removing ref from element props.
Leverages ComponentPropsWithoutRef which is a wrapper around IntrinsicElements which picks component properties from it.

@ChristianMurphy ChristianMurphy added the ☂️ area/types This affects typings label Dec 31, 2021
@ChristianMurphy ChristianMurphy requested review from a team and remcohaszing December 31, 2021 21:40
@github-actions github-actions Bot added 👋 phase/new Post is being triaged automatically 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Dec 31, 2021
@codecov-commenter

This comment has been minimized.

@ChristianMurphy ChristianMurphy force-pushed the types/remove-ref-from-instrinsic-reflection branch from b3c04c5 to 9800c41 Compare December 31, 2021 21:42
@ChristianMurphy ChristianMurphy linked an issue Dec 31, 2021 that may be closed by this pull request
4 tasks
@ChristianMurphy ChristianMurphy added the 👶 semver/patch This is a backwards-compatible fix label Dec 31, 2021
@wooorm wooorm changed the title types: remove ref from html attributes reflected from instrinsic elements Fix ref in types Jan 2, 2022
@wooorm wooorm merged commit 656a4fa into remarkjs:main Jan 2, 2022
@wooorm wooorm added the 💪 phase/solved Post is done label Jan 2, 2022
@github-actions

This comment has been minimized.

@github-actions github-actions Bot removed the 🤞 phase/open Post is being triaged manually label Jan 2, 2022
@wooorm

wooorm commented Jan 2, 2022

Copy link
Copy Markdown
Member

Released in 7.1.2!

@ChristianMurphy ChristianMurphy deleted the types/remove-ref-from-instrinsic-reflection branch January 2, 2022 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

☂️ area/types This affects typings 💪 phase/solved Post is done 👶 semver/patch This is a backwards-compatible fix

Development

Successfully merging this pull request may close these issues.

TypeError when using react-syntax-highlighter

4 participants