Debugging nodejs.fix #27893
Open
Debugging nodejs.fix #27893
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
Content: NodeJS
MaoShizhong
requested changes
Apr 29, 2024
There was a problem hiding this comment.
Thanks @MarkoMTP. Just a couple of changes required which are detailed below.
For future PRs, please could you make sure you fill out all required parts of the PR form.
| @@ -18,7 +18,7 @@ When asking a question, it is essential to provide your code, error message, ter | |||
|
|
|||
| If your question does not provide any code or other relevant information, it will be difficult for others to help you. In order to fully understand your question, there will be a lot of unnecessary back and forth conversation. This can be discouraging for you because any answers with incomplete information will not solve your problem. This process can be frustrating for those helping because the meaning of your original question changes as you add more context to it. If you truly want to ask a conceptual question, you should indicate this as part of your question. | |||
|
|
|||
| #### Ask about the problem at hand, not the solution itself | |||
| #### Ask about the problem at hand, not the solution itself, this will be edited for practice | |||
There was a problem hiding this comment.
Suggested change
| #### Ask about the problem at hand, not the solution itself, this will be edited for practice | |
| #### Ask about the problem at hand, not the solution itself |
Ideally, please make practice changes and PRs in your own test repos, otherwise you risk accidentally including unwanted changes in your PR.
| @@ -12,9 +13,10 @@ This section contains a general overview of topics that you will learn in this l | |||
|
|
|||
| <div class="lesson-content__panel" markdown="1"> | |||
|
|
|||
| - Watch [this video tutorial about Node.js debugging in VS Code](https://www.youtube.com/watch?v=2oFKNL7vYV8&ab_channel=VisualStudioCode) to see the process in action. | |||
| - Watch [video tutorial about Node.js debugging in VS Code](https://www.youtube.com/watch?v=2oFKNL7vYV8&ab_channel=VisualStudioCode) to see the process in action. | |||
There was a problem hiding this comment.
Suggested change
| - Watch [video tutorial about Node.js debugging in VS Code](https://www.youtube.com/watch?v=2oFKNL7vYV8&ab_channel=VisualStudioCode) to see the process in action. | |
| - Watch this [video tutorial about Node.js debugging in VS Code](https://www.youtube.com/watch?v=2oFKNL7vYV8&ab_channel=VisualStudioCode) to see the process in action. |
Still need this, just outside of the link text
Comment on lines
+18
to
+19
| Read this article for a tutorial on [how to debug Node with Chrome](https://web.archive.org/web/20230923074524/https://www.section.io/engineering-education/debug-node-devtools/). | ||
| NOTE: You can ignore anything that refers to debugging remote apps- you'll be debugging local apps. |
There was a problem hiding this comment.
Suggested change
| Read this article for a tutorial on [how to debug Node with Chrome](https://web.archive.org/web/20230923074524/https://www.section.io/engineering-education/debug-node-devtools/). | |
| NOTE: You can ignore anything that refers to debugging remote apps- you'll be debugging local apps. | |
| - Read this article for a tutorial on [how to debug Node with Chrome](https://web.archive.org/web/20230923074524/https://www.section.io/engineering-education/debug-node-devtools/). NOTE: You can ignore anything that refers to debugging remote apps- you'll be debugging local apps. | |
Need to keep these on the same line as well as make sure we still have the - to mark it as an item in a ul. Also popping a blank line after it since it'll fix one linting error - might as well while we're at it.
Comment on lines
+75
to
+76
| 1. [MDN's "The Server Side"](https://developer.mozilla.org/en-US/docs/Learn/Server-side/First_steps) is a great source | ||
| for the background knowledge you need. Read through at least the first two articles posted under the 'Guides' section: Introduction to the server side and Client-Server Overview. The other two are interesting and worth reviewing, but less relevant to our immediate concerns. |
There was a problem hiding this comment.
Suggested change
| 1. [MDN's "The Server Side"](https://developer.mozilla.org/en-US/docs/Learn/Server-side/First_steps) is a great source | |
| for the background knowledge you need. Read through at least the first two articles posted under the 'Guides' section: Introduction to the server side and Client-Server Overview. The other two are interesting and worth reviewing, but less relevant to our immediate concerns. | |
| 1. [MDN's "The Server Side"](https://developer.mozilla.org/en-US/docs/Learn/Server-side/First_steps) is a great source for the background knowledge you need. Read through at least the first two articles posted under the 'Guides' section: Introduction to the server side and Client-Server Overview. The other two are interesting and worth reviewing, but less relevant to our immediate concerns. |
| 1. [MDN's "The Server Side"](https://developer.mozilla.org/en-US/docs/Learn/Server-side/First_steps) is a great source | ||
| for the background knowledge you need. Read through at least the first two articles posted under the 'Guides' section: Introduction to the server side and Client-Server Overview. The other two are interesting and worth reviewing, but less relevant to our immediate concerns. | ||
| 2. To gain a little more insight into the nature of Node, and to unpack the rest of the above definition, read ["What exactly is Node.js?"](https://medium.freecodecamp.org/what-exactly-is-node-js-ae36e97449f5) | ||
| 3. What is the Node Event Loop?Check out this long, but _really_ fantastic [video on what the Node Event Loop is](https://www.youtube.com/watch?v=8aGhZQkoFbQ). Don’t skip it! |
There was a problem hiding this comment.
Suggested change
| 3. What is the Node Event Loop?Check out this long, but _really_ fantastic [video on what the Node Event Loop is](https://www.youtube.com/watch?v=8aGhZQkoFbQ). Don’t skip it! | |
| 3. What is the Node Event Loop? Check out this long, but *really* fantastic [video on what the Node Event Loop is](https://www.youtube.com/watch?v=8aGhZQkoFbQ). Don’t skip it! |
| for the background knowledge you need. Read through at least the first two articles posted under the 'Guides' section: Introduction to the server side and Client-Server Overview. The other two are interesting and worth reviewing, but less relevant to our immediate concerns. | ||
| 2. To gain a little more insight into the nature of Node, and to unpack the rest of the above definition, read ["What exactly is Node.js?"](https://medium.freecodecamp.org/what-exactly-is-node-js-ae36e97449f5) | ||
| 3. What is the Node Event Loop?Check out this long, but _really_ fantastic [video on what the Node Event Loop is](https://www.youtube.com/watch?v=8aGhZQkoFbQ). Don’t skip it! | ||
|
|
||
| 4. Take a few minutes to go through the "Getting Started" section of the new official [Node.js website](https://nodejs.org/en/learn/getting-started/introduction-to-nodejs). Read up until, but not including, the TypeScript module. |
There was a problem hiding this comment.
Suggested change
| 4. Take a few minutes to go through the "Getting Started" section of the new official [Node.js website](https://nodejs.org/en/learn/getting-started/introduction-to-nodejs). Read up until, but not including, the TypeScript module. | |
| 4. Take a few minutes to go through the ["Getting Started" section of the new official Node.js website](https://nodejs.org/en/learn/getting-started/introduction-to-nodejs). Read up until, but not including, the TypeScript module. |
This wasn't in the spreadsheet because I missed it. Just noticed it can be improved so if you wouldn't mind fixing this up too, that'd be wonderful.
|
@MarkoMTP just checking in if you've seen the above |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Because
This PR
Issue
Closes #XXXXX
Additional Information
Pull Request Requirements
location of change: brief description of changeformat, e.g.Intro to HTML and CSS lesson: Fix link textBecausesection summarizes the reason for this PRThis PRsection has a bullet point list describing the changes in this PRIssuesection