fix: Add mais validações de parâmetro na função aleatorioEntre

eguatech / egua

Linguagem de programação em português, simples e moderna

https://egua.dev/idegua/

MIT License

817 stars 40 forks source link

fix: Add mais validações de parâmetro na função aleatorioEntre #143

Closed barbozafernando closed 2 years ago

barbozafernando commented 2 years ago

Add mais validações de parâmetro para a função aleatórioEntre()
Add possibilidade da função aleatorioEntre() receber apenas um parâmentro
Add browserify às dependências do projeto, assim, cada contribuidor não precisar instalar o browserify de maneira global no seu sistema, ele executa apenas o npm install e o browserify será instalado localmente, apenas no projeto do Égua.
Add instrução para instalar as dependências do projeto no CONTRIBUTING.md

barbozafernando commented 2 years ago

@lucaspompeun

lucaspompeun commented 2 years ago

@barbozafernando já iniciei a validação, só está demorando devido ao browserify. Estou testando se a instalação dele apenas no projeto funciona, já que ele foi desenvolvido pensado em ser uma aplicação de execução global. Até o fim do dia espero ter concluído tudo.

lucaspompeun commented 2 years ago

@barbozafernando não consegui encontrar uma maneira de executar o browserify apenas no projeto, só consegui fazer funcionar com ele global mesmo. Você conhece alguma maneira de fazer isso? Se sim, me informa que consigo mergear logo o PR, caso não, vai ser preciso remover ele como dependência instalável via NPM.

barbozafernando commented 2 years ago

Sim, @lucaspompeun. Eu adicionei ele ao projeto, pois assim, qualquer um que clonar o repositório, basta executar o comando npm install pelo terminal na pasta raiz do projeto, que o browserify será instalado automaticamente. Evitando assim, da pessoa instalar uma lib global sendo que, talvez, irá usar apenas no Égua. Após isso, é só rodar o comando do npm run build-web pra ver se está funcionando.

lucaspompeun commented 2 years ago

Tudo funcionou conforme esperado com os testes. Irei aprovar o PR. Valeu demais.

lucaspompeun commented 2 years ago

@barbozafernando a pipeline irá falhar, mas irei corrigir ainda hoje para que o código entre na produção.

lucaspompeun commented 2 years ago

@barbozafernando pipeline corrigida, versão gerada e já está na produção tanto na IDEgua quanto via NPM