-
Notifications
You must be signed in to change notification settings - Fork 521
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
Note about file path source maps and some cleanup #3040
Conversation
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.
started Writer/Editor review
microsoft-edge/devtools-guide-chromium/javascript/source-maps.md
Outdated
Show resolved
Hide resolved
microsoft-edge/devtools-guide-chromium/javascript/source-maps.md
Outdated
Show resolved
Hide resolved
microsoft-edge/devtools-guide-chromium/javascript/source-maps.md
Outdated
Show resolved
Hide resolved
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.
Entered comments for Writer/Review pass.
Co-authored-by: Michael Hoffman <[email protected]>
… into user/pabrosse/enable-source-map
microsoft-edge/devtools-guide-chromium/javascript/source-maps.md
Outdated
Show resolved
Hide resolved
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.
entered comments, as a probably final Writer/Editor review pass
Learn Build status updates of commit c7d4c3e: ✅ Validation status: passedFor more details, please refer to the build report. For any questions, please:
|
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.
Approved per Writer/Editor review
Rendered articles for review:
This PR adds a short section about a new preference that needs to be enabled when using file path source maps.
The PR also contains some general updating of the source maps article, including moving out the
sourceURL
section that doesn't have anything to do with the source maps. I moved this section to the reference article instead.This PR fixes #3019.
AB#48735772