Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

checking in a trivia addition to open-sauced/pizza-verse repo's "fact… #88

Closed
wants to merge 1 commit into from
Closed

checking in a trivia addition to open-sauced/pizza-verse repo's "fact… #88

wants to merge 1 commit into from

Conversation

prw760
Copy link
Contributor

@prw760 prw760 commented May 12, 2024

…s & trivia" page.

Description

I added a fun fact about pizza and a pizza trivia item to the associated page.

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

  • [ X] ☝️ Add a pizza fact or trivia
  • 🧑‍🍳 Add a pizza recipe
  • 🗺️ Add a regional pizza
  • 📝 Documentation Update

Related Tickets & Documents

Related / Contributes to issue #36

Added to documentation?

  • 📜 README.md
  • [ X] 🙅 no documentation needed

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

image

@prw760 prw760 requested a review from a team May 12, 2024 03:19
Copy link
Member

@adiati98 adiati98 left a comment

Choose a reason for hiding this comment

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

Hey @prw760,

Thank you for your contribution! ✨

However, I see that you've added 3 items in this file.
Please read the README carefully and fix them accordingly.

Also, please change the title of the PR to follow our convention and make it short and clear. For example: feat: Add a pizza trivia.

Let me know if you have any questions or need help.
Thank you. 🙂

@prw760 prw760 closed this May 13, 2024
@prw760 prw760 deleted the doc/add-pizza-fact branch May 13, 2024 19:04
@prw760
Copy link
Contributor Author

prw760 commented May 13, 2024 via email

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.

2 participants