[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-10-04 Thread Pablo Galindo Salgado
Pablo Galindo Salgado added the comment: New changeset c4e9ef1f1d3bb7aca09f3ac6691a78d1341d7fcb by Pablo Galindo (Miss Islington (bot)) in branch '3.10': bpo-45217: adds note that `allow_no_value` in `configparser` is optional (GH-28396) (GH-28418) https://github.com/python/cpython/commit/c4

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread Łukasz Langa
Łukasz Langa added the comment: Thanks for the patch, Nikita! ✨ 🍰 ✨ -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ _

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread Łukasz Langa
Łukasz Langa added the comment: New changeset a10726d3141d8f52a108c4daf70eefa29401e2fc by Miss Islington (bot) in branch '3.10': bpo-45217: adds note that `allow_no_value` in `configparser` is optional (GH-28396) (GH-28418) https://github.com/python/cpython/commit/a10726d3141d8f52a108c4daf70e

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread Łukasz Langa
Łukasz Langa added the comment: New changeset 3ea1c4b66887e7ca920db487f6ffc5f1db3c873f by Miss Islington (bot) in branch '3.9': bpo-45217: adds note that `allow_no_value` in `configparser` is optional (GH-28396) (GH-28416) https://github.com/python/cpython/commit/3ea1c4b66887e7ca920db487f6ffc

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread miss-islington
Change by miss-islington : -- pull_requests: +26828 pull_request: https://github.com/python/cpython/pull/28418 ___ Python tracker ___ __

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread miss-islington
Change by miss-islington : -- nosy: +miss-islington nosy_count: 4.0 -> 5.0 pull_requests: +26827 pull_request: https://github.com/python/cpython/pull/28416 ___ Python tracker _

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread Łukasz Langa
Łukasz Langa added the comment: New changeset cb07838ab756564988b1ffd23871f1222a832446 by Nikita Sobolev in branch 'main': bpo-45217: adds note that `allow_no_value` in `configparser` is optional (GH-28396) https://github.com/python/cpython/commit/cb07838ab756564988b1ffd23871f1222a832446 --

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread Łukasz Langa
Łukasz Langa added the comment: No, it shouldn't be the default. It never was before (this is a library that we've maintained for 20+ years now) and changing it now would mean that existing applications would stop validating their configuration in the way they did up to now. .ini files are u

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-17 Thread sbougnoux
sbougnoux added the comment: Thanks for your prompt answer. Don't you think it should be the default? It seems like unneeded precision. Just if you want to check one don't miss a value, but in that case, one can use 'allow_no_value=False'. -- ___

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-16 Thread Nikita Sobolev
Change by Nikita Sobolev : -- keywords: +patch pull_requests: +26809 stage: -> patch review pull_request: https://github.com/python/cpython/pull/28396 ___ Python tracker ___ _

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-16 Thread Nikita Sobolev
Nikita Sobolev added the comment: Try this: ``` from configparser import ConfigParser ConfigParser(allow_no_value=True).read('bug.ini') ``` It should work! :) But, the docs are missing this config value here (which caused this issue): > Values can be omitted, in which case the key/value del

[issue45217] ConfigParser does not accept "No value" reversely to the doc

2021-09-16 Thread sbougnoux
New submission from sbougnoux : Just the simple following config crashes """ [Bug] Here """ Hopefully using "Here=" solves the issue, but the doc claims it shall work. https://docs.python.org/3.8/library/configparser.html?highlight=configparser#supported-ini-file-structure Save the config in