Skip to content

New CMU Eats README banner icon#829

Open
cirex-web wants to merge 3 commits into
stagingfrom
cirex-web-patch-1
Open

New CMU Eats README banner icon#829
cirex-web wants to merge 3 commits into
stagingfrom
cirex-web-patch-1

Conversation

@cirex-web
Copy link
Copy Markdown
Collaborator

@cirex-web cirex-web commented Apr 22, 2026

so I uh

laasyaaki and others added 2 commits April 19, 2026 14:53
Added an image to the README file.
Copilot AI review requested due to automatic review settings April 22, 2026 00:15
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cmueats Ready Ready Preview Apr 22, 2026 0:19am

Request Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a banner/icon image to the project README.

Changes:

  • Inserted a header banner image directly beneath the # CMUEats title.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
Comment thread README.md Outdated
@@ -1,4 +1,5 @@
# CMUEats
<img width="534" alt="image" src="https://github.com/user-attachments/assets/85706f1d-9509-4ac4-b9fd-0501346db526" />
Copy link

Copilot AI Apr 22, 2026

Choose a reason for hiding this comment

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

This README currently uses Markdown image syntax elsewhere (e.g., the existing images later in the file), but this new banner is embedded via raw HTML and points to a GitHub user-attachments URL. For consistency and long-term stability, consider switching to Markdown image syntax and/or checking the banner asset into the repo (e.g., under /public) so the README doesn’t depend on an external attachment link.

Suggested change
<img width="534" alt="image" src="https://github.com/user-attachments/assets/85706f1d-9509-4ac4-b9fd-0501346db526" />
![image](https://github.com/user-attachments/assets/85706f1d-9509-4ac4-b9fd-0501346db526)

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator Author

@cirex-web cirex-web Apr 22, 2026

Choose a reason for hiding this comment

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

doesn't include dimensions then

Copy link
Copy Markdown
Contributor

@Arom1a Arom1a left a comment

Choose a reason for hiding this comment

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

Why is there a femboy 😭 😭 😭

@cirex-web
Copy link
Copy Markdown
Collaborator Author

funny 😭

@cirex-web
Copy link
Copy Markdown
Collaborator Author

Other image recommendations?

@cirex-web
Copy link
Copy Markdown
Collaborator Author

cirex-web commented Apr 22, 2026

image

@cirex-web
Copy link
Copy Markdown
Collaborator Author

guess we can also flip it

image

@cirex-web
Copy link
Copy Markdown
Collaborator Author

cirex-web commented Apr 22, 2026

or this

image

@Arom1a
Copy link
Copy Markdown
Contributor

Arom1a commented Apr 22, 2026

The second one with ikuyo on the right looks great! except maybe don't make her monochrome....?

@cirex-web
Copy link
Copy Markdown
Collaborator Author

image image

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.

4 participants