Init :: Initialize git repo and new flag --noGit
#192
Conversation
nmigueles
changed the title
--git flag--noGit
|
@fdodino actualizado teniendo en cuenta lo que hablamos el miercoles! |
There was a problem hiding this comment.
Fenómeno! Te dejé una sugerencia pava de agregarle un iconito.
Por mí sale con fritas, @PalumboN @ivojawer hablamos el miércoles pasado sobre
- cambiarle el shortcut para que no se repita con el de game
- usar noGit para cambiar la opción por defecto que es sí utilizar git
- la opción de que vos pases el repo y te lo clone me parece que tenemos que iterarla, porque no me queda claro qué agregaría a lo que ya te da el git clone, o bueno, pushearlo de una puede ser pero para una v2
| @@ -61,6 +61,7 @@ program.command('init') | |||
| .option('-g, --game', 'adds a game program to the project', false) | |||
| .option('-t, --noTest', 'avoids creating a test file', false) | |||
| .option('-c, --noCI', 'avoids creating a file for CI', false) | |||
| .option('-ng, --noGit', 'avoids initializing a git repository', false) | |||
| @@ -47,6 +48,8 @@ describe('testing init', () => { | |||
| expect(join(project, 'README.md')).to.pathExists | |||
| expect(join(project, '.gitignore')).to.pathExists | |||
| expect(join(project, 'mainExample.wpgm')).to.pathExists | |||
| expect(join(project, '.git')).to.pathExists | |||
There was a problem hiding this comment.
bien ahí chequeando el directorio en los tests!
| expect(join(project, '.git')).not.to.pathExists | ||
| expect(join(project, '.git/HEAD')).not.to.pathExists |
Co-authored-by: Fernando Dodino <fernando.dodino@gmail.com>
Yo entendía que esa opción era para linkear un proyecto nuevo a un repo remoto vacío (algo que no era fácil de hacer en Xtext, siempre recomendábamos usar un template). Pero lo dejamos para otra iteración, mergiemos esto 😄 |
New option for
initcommand that initializes a git repository in the freshly created projectIf you want to opt out you can pass the --noGit or -ng option.
Usage example:
This option runs the
git initcommand.Something that i didn't test is what happens if the user does not have git installed locally.Tested and handledFixes #122