[EPIC] Commands enhancements #115
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #115 +/- ##
===========================================
+ Coverage 73.13% 91.28% +18.14%
===========================================
Files 3 5 +2
Lines 309 459 +150
Branches 82 107 +25
===========================================
+ Hits 226 419 +193
+ Misses 83 40 -43 ☔ View full report in Codecov by Sentry. |
fdodino
force-pushed
the
commands-enhancements
branch
from
1b57f6e to
a52f71c
Compare
src/index.ts
Outdated
| @@ -40,6 +43,7 @@ program.command('repl') | |||
| .option('-p, --project [filter]', 'path to project', process.cwd()) | |||
| .option('--skipValidations', 'skip code validation', false) | |||
| .option('--darkMode', 'dark mode', false) | |||
| .option('--noDiagram', 'avoid starting the server for the dynamic diagram', false) | |||
There was a problem hiding this comment.
Me costó estos parámetros en negativo. Por qué no tenerlos en positivo y con true como default?
There was a problem hiding this comment.
No es que no considere que no tenés razón... 😛
There was a problem hiding this comment.
Mmm... pero no me convence tener que decirle --diagram false para que no ejecute. El skipValidations ya estaba, no te convence si le ponemos skipDiagram? Yo se que es medio feucho, pero te propongo que si en un par de meses nos jode hagamos el refactor...
| @@ -48,6 +52,11 @@ program.command('repl') | |||
| program.command('init') | |||
| .description('Create a new Wollok project') | |||
| .option('-p, --project [filter]', 'path to project', process.cwd()) | |||
| .option('-n, --name [name]', 'name of the example', undefined) | |||
There was a problem hiding this comment.
| .option('-n, --name [name]', 'name of the example', undefined) | |
| .option('-n, --name [name]', 'name of the project', undefined) |
O a qué te referís con example?
There was a problem hiding this comment.
Es que el project tiene un path, es raro que le pongas otro nombre...
There was a problem hiding this comment.
igual tenés razón, qué onda "name of the project example file"
| else if(problems.some(_ => _.level === 'error')) { | ||
| throw new Error('Aborting run due to validation errors!') | ||
| } |
There was a problem hiding this comment.
Esto queremos tenerlo en esta función?
TENGO una propuesta:
- Que siempre se "validen" los programas -> logguear los problemas que tenga
- Que el
skipValidationssolamente (no) aborte en caso de errores
There was a problem hiding this comment.
me gusta la idea pero es algo que ya estaba, y preferiría: 1. charlarlo con más gente, 2. no meterlo dentro de este PR porque eso implica volverme a sentar a revisar los tests, y hacer tests manuales...
Cosas sueltas que quiero arreglar aprovechando la próxima versión de wollok-ts-cli
cmdhace que no se reconozcan los tests #109, arreglado en un PR de Wollok LSP IDEbuild.ymlpara github actionsAl REPL hay que
test.ts, que haya funciones más chicas para que sea más fácil testearlo:ddespués (esto lo desacoplé utilizando una interfaz que permita definir el evento onReload indicando si hay que activar el diagrama):dque en realidad solo recarga el diagrama. Ahora si no estaba levantado el diagrama lo levanta en el puerto que vos le pasaste (por defecto es el 3000, si querés levantarlo en otro puerto tenés que indicarlo antes)404la página http://localhost:3000, tengo que investigar cómo se hace esoY de paso, mejorar la cobertura de tests.
utilsinitque hoy no está cubierto (y el % no los considera)Para otro PR dejamos:
runque hoy no está cubierto (y el % no los considera)