-
-
Notifications
You must be signed in to change notification settings - Fork 90
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
fix: Update intro.md #188
fix: Update intro.md #188
Conversation
Eliminate extraneous '>' from 2nd paragraph of "Prerequisites". ("starting> this course" --> "starting this course").
✅ Deploy Preview for sauced-intro ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @georgepence,
Thank you for your contribution! ✨
However, please read our contributing guidelines.
- You need to work on existing issue or open a new issue that we've triaged before you work on your changes. That said, please create an issue and link it to your PR template.
- Please complete the PR template. It's very important for us to be able to review your changes.
I will put this PR on draft for now. Please let me know once you've addressed above suggestions, I'll change this PR back for ready to review.
Thank you! 🙂
Hey @georgepence, You now have an issue to accompany this PR (#192). Please comment on the issue with FYI, if the issue is not assigned, it's considered open for anyone to work on. |
@georgepence I see that you've assigned yourself to the issue. 👏🏼 Now, you still need to complete the PR template. You can read this section in our course to do so. I make this PR to ready for review so you can address each failed check in the checks below. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Compliance Checks Passed!
Please review changes. |
@georgepence you still have a check that needs to be addressed here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done and thank you for your contribution, @georgepence! ✨
If you haven't, you can join our community Discord.
Also, check out our pizza-verse repo if you want to contribute further! 🍕🍴
Eliminate extraneous '>' from 2nd paragraph of "Prerequisites". (replace "starting> this course" with "starting this course").
Description
This PR removes an extraneous '>', an apparent typo, from the 2nd paragraph of "Prerequisites"
Related Tickets & Documents
Fixes #192
Mobile & Desktop Screenshots/Recordings
Original (branch main):
PR will fix to display:
Steps to QA
QA process:
Tier (staff will fill in)
[optional] What gif best describes this PR or how it makes you feel?