Skip to content

fix(maintainers): move 'linkedin' URLs from 'website' to 'linkedIn' field#781

Open
sukvvon wants to merge 2 commits intoTanStack:mainfrom
sukvvon:fix/maintainers-move-linkedin-url-to-linkedIn-field
Open

fix(maintainers): move 'linkedin' URLs from 'website' to 'linkedIn' field#781
sukvvon wants to merge 2 commits intoTanStack:mainfrom
sukvvon:fix/maintainers-move-linkedin-url-to-linkedIn-field

Conversation

@sukvvon
Copy link
Contributor

@sukvvon sukvvon commented Mar 25, 2026

Move LinkedIn URLs from website to linkedIn field for Jonghyeon Ko and Arnoud de Vries so their LinkedIn icons are rendered correctly.

Summary by CodeRabbit

  • Chores
    • Corrected maintainer profile information for improved accuracy. Two maintainer entries have been updated to properly identify and display LinkedIn profiles instead of generic website references. These corrections ensure better clarity in how contributor social media profiles are represented across the project.

@netlify
Copy link

netlify bot commented Mar 25, 2026

👷 Deploy request for tanstack pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit c28ee19

@coderabbitai
Copy link

coderabbitai bot commented Mar 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3bd4eae0-d3a8-42de-93d2-8d6fb1728421

📥 Commits

Reviewing files that changed from the base of the PR and between 5922104 and 4a6b6e1.

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

📝 Walkthrough

Walkthrough

Two maintainer entries in the allMaintainers constant update their social object field nomenclature, replacing website with linkedIn to properly reflect LinkedIn profile URLs for Jonghyeon Ko and Arnoud de Vries.

Changes

Cohort / File(s) Summary
Maintainer metadata updates
src/libraries/maintainers.ts
Two maintainer entries rename the social.website field to social.linkedIn for appropriate LinkedIn profile references.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through social links so true,
From websites broad to LinkedIn's focused view,
Two maintainers' paths now clearly align,
With profiles polished—a detail so fine! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix(maintainers): move 'linkedin' URLs from 'website' to 'linkedIn' field' accurately describes the main change: moving LinkedIn URLs from one field to another for specific maintainers.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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.

1 participant