Specialize syntax error for assignment expressions in invalid places
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Feature or enhancement
Proposal:
There are a few places where (unparenthesized) assignment expressions are not accepted but users might think of using them. For example:
>>> func(arg=x := foo())
File "<python-input-0>", line 1
func(arg=x := foo())
^^
SyntaxError: invalid syntax
I find this error pretty confusing. It's not clear why this is a syntax error or what users should do it fix it (i.e. add parentheses).
I propose adding a specialized syntax error to point users in the right direction, similar to what's currently done for assert:
>>> assert x := 1
File "<python-input-12>", line 1
assert x := 1
^^^^^^
SyntaxError: cannot use named expression without parentheses here
Refs #138716, #146260
Has this already been discussed elsewhere?
This is a minor feature, which does not need previous discussion elsewhere
Links to previous discussion of this feature:
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Beispiele im issue nachzustellen und die Syntaxfehlerbehandlung von CPython für assignment expressions zu lesen, wobei du sie mit der bestehenden spezialisierten Diagnose für assert vergleichst. Als abgeschlossen gilt die Aufgabe, wenn ungültige nicht eingeklammerte assignment expressions eine klare Meldung erhalten, die Benutzer auf Klammern hinweist, während gültige Formen ihr bisheriges Verhalten beibehalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- compilers
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 58/100