camelot icon indicating copy to clipboard operation
camelot copied to clipboard

[MRG] Fix issues in utils.py, code optimisations, readability improvements, improvement in unit test coverage.

Open python3-dev opened this issue 11 months ago • 2 comments

CHANGES

utils.py

  • Fix import-error from pdfminer
  • Add type-annotations
  • Code optimisations random_string(), remove_extra(), text_strip(), compute_whitespace()
  • Improve code-readability.
  • Add more unit-test cases.

@vinayak-mehta @bosd: All tests are passing.

python3-dev avatar Jan 01 '25 15:01 python3-dev

@python3-dev Thanks for you contribution. Tests are failling sadly. Can you look into this :pray:

bosd avatar Mar 04 '25 18:03 bosd

@python3-dev the failed test for python 3.8 is due to syntax that was first supported in 3.9 (see here). Possible solutions that involve cutting back on the syntax here

DoomedJupiter avatar Mar 11 '25 01:03 DoomedJupiter