Skip to content
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

fix: Enable InvalidFileLink check when custom hrefGenerator is used #8845

Conversation

filzrev
Copy link
Contributor

@filzrev filzrev commented Jun 6, 2023

This PR resolve following problem.
When custom hrefGenerator is used.
InvalidFileLink warning check is always skipped.

This PR add additional check that hrefGenerator returns new href or not.
If href is not modified. then execute normal InvalidFileLink check path.

@codecov
Copy link

codecov bot commented Jun 6, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.01 ⚠️

Comparison is base (11161f4) 77.00% compared to head (1f62eb6) 77.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8845      +/-   ##
==========================================
- Coverage   77.00%   77.00%   -0.01%     
==========================================
  Files         605      605              
  Lines       25033    25036       +3     
==========================================
+ Hits        19277    19279       +2     
- Misses       5756     5757       +1     
Impacted Files Coverage Δ
...crosoft.DocAsCode.Build.Engine/LinkPhaseHandler.cs 98.70% <100.00%> (+<0.01%) ⬆️
....Build.TableOfContents/TocDocumentProcessorBase.cs 92.20% <100.00%> (+0.20%) ⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor

@yufeih yufeih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thank you @filzrev!

@yufeih yufeih merged commit edd5033 into dotnet:main Jun 6, 2023
@filzrev filzrev deleted the enable-filelink-warning-when-customhrefgenerator-used branch June 17, 2023 07:42
@yufeih yufeih added the bug-fix Makes the pull request to appear in "Bug Fixes" section of the next release note label Jun 20, 2023
p-kostov pushed a commit to ErpNetDocs/docfx that referenced this pull request Jun 28, 2024
…otnet#8845)

fix: enable InvalidFileLink warnings. if custom href generator returns original href
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix Makes the pull request to appear in "Bug Fixes" section of the next release note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants