Skip to content

Wendy/20260527 add food critic#418

Open
wendyxta wants to merge 1 commit into
SCE-Development:mainfrom
wendyxta:wendy/20260527_add_food_critic
Open

Wendy/20260527 add food critic#418
wendyxta wants to merge 1 commit into
SCE-Development:mainfrom
wendyxta:wendy/20260527_add_food_critic

Conversation

@wendyxta
Copy link
Copy Markdown
Contributor

merge #416 first!

@evanugarte evanugarte mentioned this pull request May 28, 2026
@thebeninator
Copy link
Copy Markdown
Contributor

there are some merge conflicts, watch the video below on how to resolve them

https://www.youtube.com/watch?v=ypiJfHblzGQ

@wendyxta wendyxta force-pushed the wendy/20260527_add_food_critic branch from 07aec9f to c416dcc Compare June 3, 2026 01:22
Comment thread dinner/texasbbq.html
Comment on lines +34 to +38
<div>
<h1> Food Critic Review </h1>
<h3> Rating: ★★★★★ </h3>
<h3> Comment: Amazing. Best brisket in all of Texas! </h3>
</div>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should be the only thing added in this diff. I see the recipe from #416 , but that same recipe is also added here which is wrong as well as your edits for index.html. Only a critic review should be added. Take a look at this pr for an example:

https://github.com/SCE-Development/git-workshop/pull/426/changes

Rebasing to your #416 branch should be the fix here.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like #416 was merged, rebase this to main

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i made the fix

@wendyxta wendyxta force-pushed the wendy/20260527_add_food_critic branch 2 times, most recently from 5074124 to cb8f9c3 Compare June 3, 2026 06:33
@wendyxta wendyxta force-pushed the wendy/20260527_add_food_critic branch from cb8f9c3 to 50fe04b Compare June 3, 2026 06:49
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