Skip to content

Conversation

@Parship999
Copy link

Which problem is this PR solving?

Description of the changes

  • The console warnings about "outdated JSX transform" are completely resolved with the configuration changes

How was this change tested?

  • Unit tests

Checklist

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
@Parship999 Parship999 requested a review from a team as a code owner September 24, 2025 05:07
@Parship999 Parship999 requested review from jkowall and removed request for a team September 24, 2025 05:07
@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 24, 2025

How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@codecov
Copy link

codecov bot commented Sep 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.16%. Comparing base (d97ab8b) to head (c70f454).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3097   +/-   ##
=======================================
  Coverage   98.16%   98.16%           
=======================================
  Files         257      257           
  Lines        8068     8068           
  Branches     2035     2112   +77     
=======================================
  Hits         7920     7920           
  Misses        148      148           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
@yurishkuro yurishkuro added the changelog:refactoring Internal, non-functional code improvements label Sep 24, 2025
Parship Chowdhury and others added 2 commits September 24, 2025 21:12
Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
@Parship999
Copy link
Author

Parship999 commented Sep 27, 2025

@yurishkuro To remove the console warnings about outdated JSX transform, the solution is to update all build tools to use modern JSX transform (react-jsx). So this will be a developer experience fix. The goal was to eliminate console warnings, which we achieved.
https://legacy.reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html

@yurishkuro yurishkuro merged commit 945f607 into jaegertracing:main Sep 27, 2025
11 checks passed
@Parship999 Parship999 changed the title Update to modern JSX transform Refactor: Update to modern JSX transform Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:refactoring Internal, non-functional code improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Update to modern JSX transform to remove the console warnings

2 participants