RustPython / RustPython/Parser

Consolidate parsing of `Expr::Name` in `TypeAliasName`, `MatchName`, and `Atom<Goal>`

Aberta
#98 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Linguagem predominante
Rust
Estrelas
117
Forks
38
Merge médio
4d 7h
PRs com merge (30d)
1

Descrição

We have three parser descriptions for in the LALRPOP grammar for generating Expr::Name from an Identifier.

  • TypeAliasName
  • MatchName
  • Atom<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

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece localizando as definições de gramática do LALRPOP para TypeAliasName, MatchName e Atom e, em seguida, compare como cada uma converte um Identifier em Expr::Name. Consolide essas descrições em um único identificador reutilizável e verifique se o comportamento de parsing existente permanece inalterado.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
rust
Domínio
compilers
Tipo de issue
Refatoração
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.