curriculum
curriculum copied to clipboard
Fixes #1229 to add reference to commit in cohort
:thinking: this will leave a reference to the commit from where a cohort has been created to the end of the README.
¡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í:
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:
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.
Tal vez de hecho el mejor lugar para esta metadata podría ser el mismo package.json
(del proyecto creado)?
@icarito puede estar en el package.json pero hay que avisar coaches que esta informacion esta alla entonces.
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.
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 hay algunos cambios antes que podemos hacer merge. deje en un review.
Para que recordamos ;-) Este seria el resultado final:
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.