Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump Plexus React typings to React 18 #1210

Merged
merged 1 commit into from
Feb 24, 2023

Conversation

mszabo-wikia
Copy link
Contributor

Which problem is this PR solving?

Contributes towards #998

Short description of the changes

Bump Plexus React typings to React 18 and also update the package's peerDependencies accordingly. I am not entirely sure why Dependabot was trying to update these typings separately because bumping them together appears to yield a successful build.

As per jaegertracing#998; also update the package's `peerDependencies` accordingly.
I am not entirely sure why Dependabot was trying to update these typings
separately because bumping them together appears to yield a successful
build.

Signed-off-by: Máté Szabó <mszabo@fandom.com>
@codecov
Copy link

codecov bot commented Feb 23, 2023

Codecov Report

Base: 95.37% // Head: 95.29% // Decreases project coverage by -0.08% ⚠️

Coverage data is based on head (51595c8) compared to base (ab46769).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1210      +/-   ##
==========================================
- Coverage   95.37%   95.29%   -0.08%     
==========================================
  Files         243      243              
  Lines        7571     7571              
  Branches     1898     1898              
==========================================
- Hits         7221     7215       -6     
- Misses        343      349       +6     
  Partials        7        7              
Impacted Files Coverage Δ
...mponents/TracePage/TraceStatistics/tableValues.tsx 93.79% <0.00%> (-2.07%) ⬇️

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@yurishkuro yurishkuro merged commit 951b6fc into jaegertracing:main Feb 24, 2023
@yurishkuro
Copy link
Member

🎉

Binrix pushed a commit to Binrix/jaeger-ui that referenced this pull request Apr 18, 2023
## Which problem is this PR solving?
Contributes towards jaegertracing#998

## Short description of the changes
Bump Plexus React typings to React 18 and also update the package's
`peerDependencies` accordingly. I am not entirely sure why Dependabot
was trying to update these typings separately because bumping them
together appears to yield a successful build.

Signed-off-by: Máté Szabó <mszabo@fandom.com>
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