update 2 places for int and str operations

dizzygz commented 8 months ago

Description:

Tasks solved Errors reported when run: howdoi --num-answers 2 python lambda function list comprehension

File "/home/dizzygz/myprog/python/howdoi/howdoi/howdoi.py", line 468, in _get_answer_worker
    multiple_answers = (args['num_answers'] > 1 or args['all'])
TypeError: '>' not supported between instances of 'str' and 'int'

File "/home/dizzygz/myprog/python/howdoi/howdoi/howdoi.py", line 437, in _get_answers
    final_pos = initial_pos + args['num_answers']
TypeError: unsupported operand type(s) for +: 'int' and 'str'

Links to issues solved

Pull Request type:

Bug fixes
New feature
Improvement
Refactoring
Documentation update
Security fix

How to test:

Please provide detailed instructions for testing your changes locally, including expected response/behavior.

Pull Request checklist:

[ ] Read the contributing_to_howdoi.md
[ ] Attach screenshots of expected behavior.
[x] The changes pass tests locally (nose2).
[ ] There are no linting errors (python setup.py lint).
[x] The changes don't break existing features.
[ ] Check that there are no confidential files like .env included.
[x] Request review from the maintainers.
[x] For bug fixes or changes to directory structure, make sure docs are updated.

Known bugs (if any):

If there are bugs in your current changes you can still open the PR and mention the bugs you found. Propose further changes that can help fix bugs in your current changes.

gleitz commented 8 months ago

Nice! Thanks for the PR

gleitz commented 8 months ago

Getting a strange installation error in the CI. I wonder if lxml version needs to be bumped?

gleitz commented 8 months ago

turn out lxml 5.0.0 was broken – they have since released 5.0.1. Re-ran the jobs and everything looks good.

gleitz / howdoi