Rebase#3600
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 829d3ab283
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| title: resume | ||
| description: resume | ||
| published: true | ||
| redirect: {{ site.url }}/assets/pdf/Avhirup_Search_Resume.pdf |
There was a problem hiding this comment.
Quote Liquid redirect value in resume front matter
The redirect entry is written as {{ site.url }}/assets/pdf/Avhirup_Search_Resume.pdf without quotes, which is invalid YAML because {} is parsed as a flow mapping. When Jekyll parses front matter for _pages, this can raise a YAML syntax error and stop the site build instead of publishing. Quote this value (or use a plain literal URL) so the page metadata parses correctly.
Useful? React with 👍 / 👎.
| </div> | ||
| title: Ask | ||
| description: | ||
| img: Ask, a QnA platform (similar to Quora) |
There was a problem hiding this comment.
Put Ask summary in description instead of img
This assigns prose text to img, but the projects template treats any non-empty img as an image URL and renders it directly in <img src=...>. As a result, the project tile generates a malformed URL and shows a broken thumbnail on /projects/; the blurb should be in description and img should be empty or a valid image path.
Useful? React with 👍 / 👎.
|
seems like a personal project, accidentally pushed to this repo. |
Rebase