Skip to content

Update check_links.py - #77

Closed
Avijit-roy wants to merge 1 commit into
flypythoncom:masterfrom
Avijit-roy:patch-1
Closed

Avijit-roy wants to merge 1 commit into
flypythoncom:masterfrom
Avijit-roy:patch-1

Conversation

@Avijit-roy

Copy link
Copy Markdown

refactor: enhance the performance and dependability of the link checker

For resilient requests, a retry strategy with exponential backoff was added. Before checking, remove duplicate URLs to prevent unnecessary requests.

  • Extract response processing and add type hints to the helper method Use pathlib.Path to handle files more neatly.
    Enhance the creation of sessions by configuring HTTPAdapter For server compatibility, a better backup from HEAD to GET requests Include visual cues (¹, →, ✗, ⏱,?) in the output.
    Make max_workers and timeout configurable.

refactor: enhance the performance and dependability of the link checker

For resilient requests, a retry strategy with exponential backoff was added.
Before checking, remove duplicate URLs to prevent unnecessary requests.
- Extract response processing and add type hints to the helper method
Use pathlib.Path to handle files more neatly.
Enhance the creation of sessions by configuring HTTPAdapter
For server compatibility, a better backup from HEAD to GET requests
Include visual cues (¹, →, ✗, ⏱,?) in the output.
Make max_workers and timeout configurable.
@xxg1413

xxg1413 commented Oct 4, 2026

Copy link
Copy Markdown
Member

Thanks for the effort here! Closing: the branch now conflicts with master, and check_links.py has repository rules (read-only, rate-limited, retryable, blocked from private/loopback targets — see AGENTS.md) that a full rewrite needs to be reviewed against in small steps. Small, focused PRs against the current file are welcome.

@xxg1413 xxg1413 closed this Oct 4, 2026
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