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.

Sign up for GitHub

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

Jump to bottom

Blondon1 patch 3 #371

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Blondon1 patch 3 #371

wants to merge 2 commits into from

Commits on Jan 29, 2024

  1. Update pdfToText.py 

    Improved error handling
    @blondon1
    blondon1 authored Jan 29, 2024
    Configuration menu
    Copy the full SHA
    077d46d View commit details
    Browse the repository at this point in the history

Commits on Jan 30, 2024

  1. improved for robustness, efficiency, and code quality 

    check the HTTP response status explicitly, robust parsing to check the elements exist before accessing them, also when dealing with file operations, it's always a good practice to use the with statement, Added a user-agent to the request header because some websites might block requests that do not originate from a browser
    @blondon1
    blondon1 authored Jan 30, 2024
    Configuration menu
    Copy the full SHA
    565c2ed View commit details
    Browse the repository at this point in the history