-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Project: Recipes: Add creation of backlink to recipes tutorial #29238
Conversation
add instructions for back link from recipes to homepage (index.html)
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.
Hello there! Thank you for contributing.
You checked off several items in the PR description. Here are three of them that I'd like to call out:
- The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
- If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
- The Because section summarizes the reason for this PR
Please go over these bullet points and make sure you're adhering to the guidelines you agreed to. I'd look at some of the other active or merged PRs.
Thanks!
Thank you Zach! I've addressed the issues and will update shortly. E: updated |
fix typo and use correct markdown syntax for backlink html example
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.
Ideally I'd say to match the style of the rest of the page and bullet points here, but I appreciate your emphasis and code block. Thanks for contributing!
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.
As soon as I approved it the linter found some issues, if you could update your work here and re-request, I'll can re-approve afterwards. Thanks!
- add blank line before and after fenced html - ignore capitalization, html is not used as a noun here
@Keiferton looks like the linter caught another issue. |
the linter showed an error on line 71, I believe it to be from a line break on an indented line, removed indent
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.
Thanks for all of your hard work here!
add instructions for back link from recipes to homepage (index.html)
Because
In Project: Recipes, adding instructions for how to back link to index.html, can make it so beginners can easily navigate while they are changing their pages. Tutorial doesn't explicitly recommend to do this, so this could be helpful.
This PR
Issue
Closes #XXXXX
Additional Information
Pull Request Requirements
location of change: brief description of change
format, e.g.Intro to HTML and CSS lesson: Fix link text
Because
section summarizes the reason for this PRThis PR
section has a bullet point list describing the changes in this PRIssue
section