Skip to content

fix(react): honor JSX pragma in display-name - #2095

Merged
fansenze merged 2 commits into
mainfrom
fix/react-display-name-jsx-pragma
Sep 7, 2026
Merged

fansenze merged 2 commits into
mainfrom
fix/react-display-name-jsx-pragma

Conversation

@elecmonkey

Copy link
Copy Markdown
Member

Summary

Make react/display-name resolve the React pragma from source @jsx comments before falling back to settings, matching eslint-plugin-react v7.37.5.

Restore the skipped upstream case and add 12 regression scenarios covering pragma precedence, fallback, and wrapper substitution in both source-only and checker-backed Programs.

Validation

  • Compared all 12 scenarios with eslint-plugin-react v7.37.5.
  • Go tests passed for display_name and reactutil.
  • Rebuilt the CLI; display-name JS integration tests passed.
  • Scoped Go lint, spell check, and format check passed.

Checklist

  • Tests updated.
  • Documentation updated.

@elecmonkey
elecmonkey marked this pull request as draft September 7, 2026 10:58
@elecmonkey
elecmonkey marked this pull request as ready for review September 7, 2026 11:01
@fansenze
fansenze merged commit b3aa8ae into main Sep 7, 2026
14 checks passed
@fansenze
fansenze deleted the fix/react-display-name-jsx-pragma branch September 7, 2026 11:31
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.

2 participants