shelljs/shelljs

Refactor ShellString into a real class

Open

#651 aberto em 20 de jan. de 2017

Ver no GitHub
 (9 comments) (0 reactions) (0 assignees)JavaScript (777 forks)batch import
breakinggood first issuemedium priorityrefactor

Métricas do repositório

Stars
 (14.062 stars)
Métricas de merge de PR
 (Mesclagem média 3d 12h) (2 fundiu PRs em 30d)

Description

ShellString is a huge hack right now, and isn't really implemented as a class with a prototype. This means it can't be modified at runtime, which significantly limits what plugins can do. We should refactor it into an extendable class (preferably one which inherits all string methods).

We should also consider a ShellArray class for methods like ls(), to make implementation easier.

This blocks #486

Guia do colaborador

Direção de pesquisa
Estude a implementação atual do ShellString no shell.js, entenda como é usado e refatore o em uma classe que estende String ou usa uma cadeia de protótipos para herdar métodos de string. Garanta compatibilidade com testes e plugins existentes.
Pilha de tecnologia
javascript
Domain
clitooling
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
JavaScriptNode.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50