Skip to content

exclude non-latest versions of docs from broken link checker#3401

Merged
fjlanasa merged 2 commits intomainfrom
fl/fix-link-checker
Mar 25, 2022
Merged

exclude non-latest versions of docs from broken link checker#3401
fjlanasa merged 2 commits intomainfrom
fl/fix-link-checker

Conversation

@fjlanasa
Copy link
Contributor

@fjlanasa fjlanasa commented Mar 24, 2022

Pull Request Description

Follow up to #3398

The crawler ended up following a link on the latest version of the docs to the stable version of the docs, which did not contain a fix to a broken link, leading to a failed check. This PR updates the exclude argument to not check documentation pages for versions other than latest.


After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

@fjlanasa fjlanasa force-pushed the fl/fix-link-checker branch 2 times, most recently from 6fca6a4 to ccabf8a Compare March 24, 2022 15:07
@codecov
Copy link

codecov bot commented Mar 24, 2022

Codecov Report

Merging #3401 (cb84c48) into main (ba87d3f) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #3401   +/-   ##
=====================================
  Coverage   99.6%   99.6%           
=====================================
  Files        329     329           
  Lines      32380   32380           
=====================================
  Hits       32249   32249           
  Misses       131     131           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba87d3f...cb84c48. Read the comment docs.

@fjlanasa fjlanasa force-pushed the fl/fix-link-checker branch from ccabf8a to 3f6eaca Compare March 24, 2022 17:56
Copy link
Contributor

@freddyaboulton freddyaboulton left a comment

Choose a reason for hiding this comment

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

Thank you for the fix @fjlanasa !

@fjlanasa fjlanasa merged commit ed790a0 into main Mar 25, 2022
@chukarsten chukarsten mentioned this pull request Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants