Conversation
There was a problem hiding this comment.
Pull request overview
Updates personal/education copy in the site’s “sections” content, primarily reflecting post-graduation status and updated graduation timeline.
Changes:
- Update the bio caption to reflect graduation status.
- Update WSU timeline entries from “expected graduation” to “graduated” for BS/MS terms.
- Normalize one shields.io badge string quoting in the frameworks list.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/sections/index.js | Updates bio caption and adjusts a frameworks badge URL string. |
| src/sections/University.js | Updates university timeline entries to “graduated” phrasing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 'https://img.shields.io/badge/Express.js-000000?style=for-the-badge&logo=express&logoColor=white', | ||
| 'https://img.shields.io/badge/Flask-000000?style=for-the-badge&logo=flask&logoColor=white', | ||
| "https://img.shields.io/badge/spring-%236DB33F.svg?style=for-the-badge&logo=spring&logoColor=white&textColor=black)", | ||
| 'https://img.shields.io/badge/spring-%236DB33F.svg?style=for-the-badge&logo=spring&logoColor=white&textColor=black)', |
There was a problem hiding this comment.
The Spring shields.io badge URL ends with an extra closing parenthesis (...textColor=black)), which likely makes the image request invalid and the badge fail to render. Remove the trailing ) (or verify the intended query string) so the URL is a valid shields.io endpoint.
| 'https://img.shields.io/badge/spring-%236DB33F.svg?style=for-the-badge&logo=spring&logoColor=white&textColor=black)', | |
| 'https://img.shields.io/badge/spring-%236DB33F.svg?style=for-the-badge&logo=spring&logoColor=white&textColor=black', |
| caption: | ||
| 'Zavaar Shah is a computer science student at Wayne State University.', | ||
| 'Zavaar Shah is a computer science graduate from Wayne State University.', | ||
| langs: { |
There was a problem hiding this comment.
The PR title/description indicates a "new headshot", but this diff only updates text content (bio line and graduation timeline) and does not appear to change any image/headshot assets or references. If the intent is to update the headshot, include the asset/reference change; otherwise consider updating the PR title/description to match the actual changes.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #34 +/- ##
=======================================
Coverage 68.58% 68.58%
=======================================
Files 16 16
Lines 226 226
Branches 65 65
=======================================
Hits 155 155
Misses 67 67
Partials 4 4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Out with the old, in with the new :)