[DevTools] Update ReadMe Docs For WebExtensions #34496
Closed
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.
Summary
All three browser extension ReadMe files reference a path (
shells/webextension) for common source code that I believe no longer exists.I have a guess where the correct path now is, but given it should be easy for the maintainers to make this correction themselves (accurately and without guessing), I'm simply flagging it.
Thus for this PR to be completed it would need to replace the struck out characters with an updated, correct path. If I have made a mistake I apologize, but as you can see in that screenshot I could not find
shells/webextension... I was able to find a top-levelshellsdirectory in the git history, but I must admit the git history is a little confusing, perhaps because this code used to live in a separate repo once upon a time?In any event, it would be great if one of the maintainers could spend a few minutes confirm the issue and give a quick update for people coming to this fresh who want to dive into this particular aspect of the source code. Thanks for your time!
How did you test this change?
N/A