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

Comment changes to uploader.py and textsearch.py #36

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mirthala-lopez
Copy link
Collaborator

The comments should now follow the Python protocol. @lindsaypoirier I changed some comments to be a bit more clear about what things, and I removed comments that I felt weren't needed if the line it followed was straightforward. @DexterAntonio I noticed there was an extra for loop in the 'highlight_pdf' function that would make the program go though all iterations of wordcount twice. I tested it, and removing it gave the same results. Was this okay?

…ordcount' loop.Changed and removed some comments if unclear or not needed.
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.

1 participant