Skip to content

Commit

Permalink
address more code review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
croth1 committed Dec 6, 2021
1 parent 376e3a6 commit 264051c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion sphinx/builders/linkcheck.py
Expand Up @@ -551,6 +551,7 @@ def setup(app: Sphinx) -> Dict[str, Any]:
app.add_post_transform(HyperlinkCollector)

app.add_config_value('linkcheck_ignore', [], None)
app.add_config_value('linkcheck_exclude_documents', [], None)
app.add_config_value('linkcheck_allowed_redirects', {}, None)
app.add_config_value('linkcheck_auth', [], None)
app.add_config_value('linkcheck_request_headers', {}, None)
Expand All @@ -561,7 +562,6 @@ def setup(app: Sphinx) -> Dict[str, Any]:
# Anchors starting with ! are ignored since they are
# commonly used for dynamic pages
app.add_config_value('linkcheck_anchors_ignore', ["^!"], None)
app.add_config_value('linkcheck_exclude_documents', [], None)
app.add_config_value('linkcheck_rate_limit_timeout', 300.0, None)

app.add_event('linkcheck-process-uri')
Expand Down
2 changes: 1 addition & 1 deletion tests/roots/test-linkcheck-documents_exclude/conf.py
Expand Up @@ -2,4 +2,4 @@
linkcheck_exclude_documents = [
'^broken_link$',
'br[0-9]ken_link',
]
]
2 changes: 1 addition & 1 deletion tests/test_build_linkcheck.py
Expand Up @@ -634,4 +634,4 @@ def test_linkcheck_exclude_documents(app):
with open(app.outdir / 'output.json') as fp:
for record in fp:
content = json.loads(record)
assert content['status'] == 'unchecked'
assert content['status'] == 'ignored'

0 comments on commit 264051c

Please sign in to comment.