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.
Description:
I'm once again sorry for the very large PR. This PR fixes issue where
autoLinkCode
wasn't able to produce links in code snippets. For example, if there was a snippet like this (in any exported.ts
or.md
file):autoLinkCode
would generate links whereverfrom
,Subject
andmulticast
were written becausefrom
,Subject
andmulticast
are valid documentation pages.The real reason why this
autoLinkCode
failed to do some processing was that it was actually excluded frompostProcessHtml.plugins
. At first, it was added to thepostProcessHtml.plugins
array, but later,plugins
array was overwritten with another array (with aembedMarbleDiagramsPostProcessor
plugin). Now, I fixed it to pushembedMarbleDiagramsPostProcessor
to an existing array instead of creating a new one.autoLinkCode
was adjusted so that it follows the same code structure from Angular project.Also, one more thing that
embedMarbleDiagramsPostProcessor
was doing is that it was copying files fromdocs_app/src/assets/images/marble-diagrams/
todocs_app/assets/
for no particular reason. I couldn't ever find where these files have been used, because all images in the app have hadsrc
tag that includedassets/images/marble-diagrams/
path. For that reason, I removed copying these files, I removed all copied files and I removed the dependency associated with that -mkdirp
.With removal of
mkdirp
, I noticed another package that was accidentally added that's never used -typedoc
. The docs generation is done by Angular scripts.Since these old (and reintroduced)
postProcessHtml.plugins
were never used for more than 2 years, some docs are now having multiple issues. One of the issues is that there are multiple docs where there are multiple H1 HTML tags. For that reason, I disabled a processor that checks that because it fails to generate scripts. I will try to fix those issues if this PR lands to master.One last note: while I was creating PRs related to #6627, I forgot to remove old images (
audit.png
,debounce.png
andthrottle.png
), so I removed them here.Related issue (if exists):
None, only this comment.