TheSpyder/rescript-webapi

Remove trailing _ from method and parameter names

Aberta

#29 aberto em 23 de set. de 2021

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)ReScript (38 forks)github user discovery
good first issue

Métricas do repositório

Stars
 (159 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

In many cases, ReScript syntax is unambiguous enough that a trailing _ is no longer required in places it was for ReasonML and OCaml syntax. A codebase search for _: turns up quite a lot of them. Examples: https://github.com/tinymce/rescript-webapi/blob/ea9bd980f1b2b2d516e518f1f9c166feb1073f33/src/Webapi/Webapi__Blob.res#L10

https://github.com/tinymce/rescript-webapi/blob/253ba954381a78b57bad249ea83c1f5b6ab2b428/src/Webapi/Dom/Webapi__Dom__HtmlFormElement.res#L17

The compiler will signal when removing the underscore is not allowed, making this a mostly trivial change (but some tests would be nice)

Guia do colaborador

Direção de pesquisa
Pesquise no código por padrões ` :`, remova o underscore final onde o compilador permitir e execute os testes para garantir a correção.
Pilha de tecnologia
None
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80