Skip to content

Conversation

@emilyzyzhou
Copy link
Collaborator

@emilyzyzhou emilyzyzhou commented Jan 18, 2026

Added profile picture to the about page

Summary by CodeRabbit

  • New Features

    • Added Emily Zhou to the engineering team listing.
  • Documentation

    • Updated alumni/contributors listing: moved Vivian Gao from the Class of 2025 to the Class of 2026 to reflect current membership.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

📝 Walkthrough

Walkthrough

Added a new engineering team member (Emily Zhou) to team_info.json and moved Vivian Gao from Class of 2025 to Class of 2026 in team_alums.json; no other structural changes.

Changes

Cohort / File(s) Summary
Engineering roster
tcf_website/views/team_info.json
Added one object for Emily Zhou to the engineering_team array (fields: name, role, class, img_filename, github).
Alumni / contributors
tcf_website/views/team_alums.json
Moved Vivian Gao from Class of 2025 to Class of 2026 (removed from 2025 list, added to 2026 list).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • blairezh
  • jo-hya

Poem

🐰 I hopped through JSON fields today,
A new name nested in array.
Emily joins, Vivian moves on,
Little changes, big bright dawn.
🥕 Welcome, team — let's code away!

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is minimal and missing most required template sections including GitHub issues, detailed explanation of changes, screenshots, testing details, and questions/notes. Complete the pull request description template by adding details on what was changed (Emily Zhou added, Vivian's class updated), how to test, and any relevant notes.
Title check ❓ Inconclusive The title 'added emily pfp' is vague and uses unclear abbreviation; it doesn't clearly convey the main changes to team members or alumni data. Use a more descriptive title like 'Add Emily Zhou to engineering team and update Vivian to Class 2026' to clearly summarize the main changes.
✅ Passed checks (1 passed)
Check name Status Explanation
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.


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.

@emilyzyzhou emilyzyzhou assigned Copilot and blairezh and unassigned Copilot Jan 18, 2026
@blairezh blairezh self-requested a review January 18, 2026 21:27
@blairezh blairezh removed their assignment Jan 18, 2026
Copy link
Contributor

@blairezh blairezh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image is in proper format and no issues with display on team page

@blairezh blairezh merged commit ec18e93 into dev Jan 20, 2026
4 checks passed
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.

3 participants