PEP 722/723 decision

[ofek (Ofek Lev)]

I will be updating the PEP in a few minutes but I wanted to address two things:

@h-vetinari

but even before that change, popping the current reference regex into https://regex101.com/ shows that it doesn’t work on many other engines either

That’s not exactly correct. Forward slashes have no significance in regular expressions but rather may be used as a delimiter in determining the start/end of patterns in for example the sed command and JavaScript. The regular expression is compatible with everything right now.

@ntessore

I’m not sure it’s a contrived example, since the [tool] table might contain arbitrary strings containing ///. This breaks the current ducktools parser: This seemingly reasonable config cannot currently be parsed with the implementation offered above: [1]

# /// pyproject
# [run]
# requires-python = ">=3.11"
# [tool.pep723lint]
# markers = """
# ///
# """
# ///

The reference implementation does parse this properly because the regular expression is greedy and will keep going until the final end delimiter without the next line being a comment. I will update the text to describe this semantic.