Added support for Python 3.13 (latest version) in tox.ini

[mohsinsurani]

digit_group_separator and price is not none - make sured to make all the test passed.

[Gallaecio]

We should add support for all other Python versions in between.

[mohsinsurani]

Thanks. I added Support for different versions of python

[Gallaecio]

Minor thing: could you remove the spaces after the comma? For consistency with other projects.

[mohsinsurani]

No worries. I removed spaces.

[Gallaecio]

Thanks.

I think the only things left are to update these accordingly:

• price-parser/.github/workflows/main.yml

  Lines 25 to 27 in bfd1f14

  	- python-version: "3.10"
  	env:
  	TOXENV: py

• price-parser/setup.py

  Line 28 in bfd1f14

  'Programming Language :: Python :: 3.8',

[mohsinsurani]

Thanks. I updated the code. Do I require to change for mypy?

[Gallaecio]

Do I require to change for mypy?

Great point. It should indeed run with the latest Python version ideally.

[mohsinsurani]

Thanks, I made the changes.

[Gallaecio]

https://github.com/scrapinghub/price-parser/pull/66/files#r1810447696 is not addressed yet. It’s empty space in line 40 of the new code.

[mohsinsurani]

Sorry, I missed it. I committed this change.

[Gallaecio]

I’ve just realized that I recently lost admin access to the repo 🤦

As soon as I get it back I’ll approve the CI run, and merge if all goes well. Thanks!

[Gallaecio]

OK, the only failing issue is due to 3.6 not being available in CI. We can handle dropping end-of-life Python versions in a separate PR later on. Thank you!