curriculum icon indicating copy to clipboard operation
curriculum copied to clipboard

Fixes #1229 to add reference to commit in cohort

Open icarito opened this issue 2 years ago • 3 comments

:thinking: this will leave a reference to the commit from where a cohort has been created to the end of the README.

icarito avatar Sep 23 '22 16:09 icarito

¡Mil gracias @icarito por el PR :blush: :raised_hands: !

Estoy de acuerdo con el comentario de @unjust en que quizás el comportamiento esperado es en realidad quizás poder ver el número de versión actual del repo más que el hash de un commit, que luego habría que ir a buscar.

Hice un pequeño cambio probando en mi máquina, dejándolo así: 'Bootcamp version: ' + process.env.npm_package_version

Aprovechando que el número de versión del package.json es el del release actual de la currícula, entonces al final se ve así:

Screenshot_2022-10-04_17-38-59

Voy a seguir haciendo algunas pruebitas, para ver si el lugar/momento donde está ocurriendo eso es el mejor y cómo dejar al final igual una new line vacía, que es regla de nuestro linter de archivos markdown.

¡Nuevamente mil gracias a amb@s! :smile: :yellow_heart:

mfdebian avatar Oct 04 '22 20:10 mfdebian

Gracias @mfdebian y @unjust - me pillaste sin alcanzar a hacer más cambios, era solo una prueba de concepto. Como no sabía que al versión NPM iba sincronizada (o pensaba que podría des-sincronizarse).

Te agradezco las pruebitas, de hecho no me quedaba claro donde poner metadata como esto excepto al pie del README, lo cual puede no ser lo mejor.

icarito avatar Oct 04 '22 20:10 icarito

Tal vez de hecho el mejor lugar para esta metadata podría ser el mismo package.json (del proyecto creado)?

icarito avatar Oct 04 '22 20:10 icarito

@icarito puede estar en el package.json pero hay que avisar coaches que esta informacion esta alla entonces.

unjust avatar Oct 05 '22 14:10 unjust

creo como asi puede ser:

 const addBootcampVersion = async (repoDir) => {
     const projectPkgJsonPath = path.resolve(`${repoDir}/package.json`);
     let pkg = require(projectPkgJsonPath);
     pkg.bootcamp_version = process.env.npm_package_version; 
     await fs.writeFile(projectPkgJsonPath, JSON.stringify(pkg, null, 2));
}

y alrededor linea 243:

 await copy(src, repoDir, opts);
 await addBootcampVersion(repoDir);

por algun razon no podia hacer un edicion a tu pull request.

unjust avatar Oct 07 '22 19:10 unjust

Estupendo @mfdebian y @unjust, ha sido un gusto colaborar con Uds! Creo que @mfdebian completó los cambios sugeridos y esto podría estar listo para merge.

icarito avatar Oct 26 '22 12:10 icarito

@icarito hay algunos cambios antes que podemos hacer merge. deje en un review.

unjust avatar Oct 26 '22 17:10 unjust

Para que recordamos ;-) Este seria el resultado final:

unjust avatar Oct 26 '22 17:10 unjust

Gracias @lupomontero @mfdebian e @unjust por los comentarios, el PR quedó con los cambios solicitados. :pray: Ya lo probé y funcionó como se espera a la primera.

icarito avatar Nov 05 '22 11:11 icarito