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

DRY v1 #4

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

DRY v1 #4

wants to merge 19 commits into from

Conversation

mcascone
Copy link

@mcascone mcascone commented May 7, 2024

Only the truly repetitive parts were dried. I didn't make new logic to put all extraction in one file, as it is clear there are specific paths the non-generic loaders need to take.

The local helpers.py file is imported as shown:

sys.path.append(os.path.dirname(os.path.realpath(__file__)))
from helpers import hello_from_script, log_directories, extract_text_from_all_pdfs

This assumes the helpers.py file is in the same dir as the consuming script.

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.

None yet

1 participant