RustPython / RustPython/Parser
Consolidate parsing of `Expr::Name` in `TypeAliasName`, `MatchName`, and `Atom<Goal>`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Rust
- Sterne
- 117
- Forks
- 38
- Ø Merge
- 4 T. 7 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
We have three parser descriptions for in the LALRPOP grammar for generating Expr::Name from an Identifier.
TypeAliasNameMatchNameAtom<Goal>
These could be consolidated into a single reused identifier for simplicity and clarity.
Originally posted by @MichaReiser in https://github.com/RustPython/Parser/pull/97#discussion_r1263930630
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
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 LALRPOP-Grammatikdefinitionen für TypeAliasName, MatchName und Atom zu finden, und vergleiche anschließend, wie jede davon einen Identifier in Expr::Name umwandelt. Fasse diese Beschreibungen in einem wiederverwendbaren Identifier zusammen und überprüfe, dass das bestehende Parsing-Verhalten unverändert bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- compilers
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100