Skip to content

Add since annotation to useActiveTooltipDataPoints#7338

Merged
PavelVanecek merged 1 commit into
mainfrom
useActiveTooltipDataPoints-since
May 16, 2026
Merged

Add since annotation to useActiveTooltipDataPoints#7338
PavelVanecek merged 1 commit into
mainfrom
useActiveTooltipDataPoints-since

Conversation

@PavelVanecek

@PavelVanecek PavelVanecek commented May 16, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Only docs

Summary by CodeRabbit

  • Documentation
    • Updated JSDoc comments to improve code documentation clarity.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4d63742c-b8e2-44cb-81ed-453f5fd314af

📥 Commits

Reviewing files that changed from the base of the PR and between e3a1f82 and dc500f6.

📒 Files selected for processing (1)
  • src/hooks.ts

Walkthrough

This PR updates the JSDoc comment for the useActiveTooltipDataPoints hook in src/hooks.ts to include or realign the @since 3.1 version annotation. No runtime logic or function signatures are changed.

Changes

Hook Documentation

Layer / File(s) Summary
useActiveTooltipDataPoints JSDoc annotation
src/hooks.ts		 JSDoc for useActiveTooltipDataPoints is modified to include/realign the @since 3.1 annotation within the hook documentation comment.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • recharts/recharts#7326: Both PRs only update JSDoc comments for hooks with no runtime or signature changes.
  • recharts/recharts#6927: Both PRs update JSDoc documentation in src/hooks.ts to adjust/add @since version annotations for hooks.

Suggested reviewers

  • ckifer
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete. It only contains 'Only docs' without providing the required template sections (Description, Related Issue, Motivation and Context, etc.). Provide a detailed description following the repository template, including sections for motivation, testing details, and change type classification.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and specifically describes the main change: adding a since annotation to the useActiveTooltipDataPoints hook's documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch useActiveTooltipDataPoints-since

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented May 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Bundle Report

Changes will increase total bundle size by 68 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
recharts/bundle-cjs 1.4MB 17 bytes (0.0%) ⬆️
recharts/bundle-es6 1.23MB 17 bytes (0.0%) ⬆️
recharts/bundle-treeshaking-cartesian 701.84kB 17 bytes (0.0%) ⬆️
recharts/bundle-treeshaking-polar 485.23kB 17 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: recharts/bundle-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
hooks.js 17 bytes 23.35kB 0.07%
view changes for bundle: recharts/bundle-es6

Assets Changed:

Asset Name Size Change Total Size Change (%)
hooks.js 17 bytes 21.17kB 0.08%
view changes for bundle: recharts/bundle-treeshaking-cartesian

Assets Changed:

Asset Name Size Change Total Size Change (%)
bundle.js 17 bytes 701.84kB 0.0%
view changes for bundle: recharts/bundle-treeshaking-polar

Assets Changed:

Asset Name Size Change Total Size Change (%)
bundle.js 17 bytes 485.23kB 0.0%

@chromatic-com

chromatic-com Bot commented May 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Tip

All tests passed and all changes approved!

🟢 UI Tests: 193 tests unchanged
🟢 UI Review: 193 stories published -- no changes
Storybook icon Storybook Publish: 193 stories published

@codecov

codecov Bot commented May 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.99%. Comparing base (021b02c) to head (dc500f6).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #7338   +/-   ##
=======================================
  Coverage   88.99%   88.99%           
=======================================
  Files         541      541           
  Lines       41136    41136           
  Branches     5569     5569           
=======================================
  Hits        36610    36610           
  Misses       4518     4518           
  Partials        8        8

☔ 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.

@github-actions

github-actions Bot commented May 16, 2026

Copy link
Copy Markdown
Contributor

Staging Deployment Details

These deployments will remain available for 30 days.

To update snapshots: Comment /update-snapshots on this PR to automatically update the baseline screenshots.

@PavelVanecek
PavelVanecek merged commit 4b2643d into main May 16, 2026
60 checks passed
@PavelVanecek
PavelVanecek deleted the useActiveTooltipDataPoints-since branch May 16, 2026 08:46
@coderabbitai coderabbitai Bot mentioned this pull request May 24, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@PavelVanecek