Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(Project): Adds support for variant languages/optional and turning off learning objectives #88

Merged
merged 15 commits into from
May 22, 2024

Conversation

mfdebian
Copy link
Contributor

@mfdebian mfdebian commented Mar 27, 2024

@adolivaresl dejo acá subido lo que revisamos juntxs.

para ejecutarlo, luego de traerte los cambios, hay que hacer npm link y luego, para probar, estaba ejecutando el siguiente comando:

curriculum-parser project ../curriculum/projects/05-fleet-management-api/ --repo Laboratoria/pepita --version 6.6.6.6.6.6 --lo=../curriculum/learning-objectives --variants=python,node > fleet-management-api.json

donde en el project.yml del proyecto 05-fleet-management-api nos habíamos traído los cambios de merun, los atributos --repo y --version son opcionales y puede poner lo que quieras, jijiji y luego todo ese output lo estamos arrojando a un archivo llamado fleet-management-api.json para ir revisando el output

@mfdebian mfdebian changed the title WIP-feat(Project): Adds support for variant learning objectives feat(Project): Adds support for variant learning objectives Apr 2, 2024
@mfdebian mfdebian changed the title feat(Project): Adds support for variant learning objectives feat(Project): Adds support for variant/optional and turning off learning objectives Apr 3, 2024
@mfdebian mfdebian changed the title feat(Project): Adds support for variant/optional and turning off learning objectives feat(Project): Adds support for variant languages/optional and turning off learning objectives Apr 3, 2024
@mfdebian mfdebian requested a review from merunga April 9, 2024 18:38
merunga
merunga previously approved these changes Apr 9, 2024
lib/project.js Outdated Show resolved Hide resolved
lib/__tests__/project.spec.js Show resolved Hide resolved
Copy link
Contributor

@unjust unjust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Toma mis sugerencias como sugeriencias ;-)
Para mi puedes mejorar algunos nombres y variables para alguien puede entender la forma de learningObjective y algunos casos.

lib/project.js Outdated Show resolved Hide resolved
lib/project.js Outdated Show resolved Hide resolved
lib/project.js Outdated Show resolved Hide resolved
lib/project.js Outdated Show resolved Hide resolved
lib/project.js Outdated Show resolved Hide resolved
lib/project.js Outdated Show resolved Hide resolved
lib/project.js Outdated Show resolved Hide resolved
@lupomontero lupomontero merged commit 26d16a0 into Laboratoria:main May 22, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants