search

open-sauced / pizza-verse

A repository for all things pizza.
MIT License
38 stars 55 forks source link

feat: Added Pizza Fact #96

Closed prw760 closed 3 months ago

prw760 commented 3 months ago

Description

Added a pizza fact into the repo's fact and trivia markdown file

What type of PR is this? (check all applicable)

Related Tickets & Documents

Relates to #36

Added to documentation?

[optional] What GIF best describes this PR or how it makes you feel?

adiati98 commented 3 months ago

Hey @prw760,

You have a failed check here. You need to address it before we can review your PR.

Thank you! 🙂

prw760 commented 3 months ago

Hi, Ayu. I fixed the problem with the title of the PR, and it appears to have passed the checks now. Hopefully this is merge ready, finally! Thanks again.

adiati98 commented 3 months ago

You did fix the problem, @prw760! Well done! 🎉

But can you please change the issue number? At the moment, you're referring to the closed PR number. 🙂

Also, you need to put "Relates to" and not "Fixes" in front of the issue number to keep it open as mentioned in the issue. Thanks!

prw760 commented 3 months ago

Hi, again, Ayu. I fixed the issue number (to #36) and also changed "Fixes" to "Relates to". Hopefully this works! :)

prw760 commented 3 months ago

You are super patient, Ayu! That's an incredible skill. Thanks again for all your help! Every time you have an issue with one of my PRs, I learn more. :)

Sent with Proton Mail secure email.

On Friday, May 17th, 2024 at 2:15 PM, Ayu Adiati @.***> wrote:

Merged #96 into main.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

adiati98 commented 3 months ago

And so are you, Paul! You have growth mindset which is very important to have. 🙌🏼