google/closure-compiler

I think parseInt radix should be optional

Aberta

#3.230 aberto em 12 de fev. de 2019

Ver no GitHubGuia do colaborador
 (12 comentários) (0 reação) (0 responsável)Java (1.197 forks)batch import
P4good first issueinternal-issue-createdtriage-done

Métricas do repositório

Stars
 (7.176 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

I understand why this is. But if feels like a very old legacy issue...

https://github.com/google/closure-compiler/blob/b29d09f56b56e5b3ea78ba0317ae579b837fd821/externs/es3.js#L289-L300

Nowdays all browser use radix 10 as default and has done so for a long time. And the likelihood of someone starting the number with 0 and using IE8 is very slim. Many have stop supporting old browsers

* @param {number=} base

Guia do colaborador

Direção de pesquisa
Revise a especificação ECMAScript para o comportamento padrão da base de parseInt e examine como os externs do Closure Compiler afetam os avisos. Considere as implicações de compatibilidade com navegadores e teste as alterações.
Pilha de tecnologia
javascript
Domain
backend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70