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

studentify use instructions should be clarified #63

Closed
renghen opened this issue Jul 14, 2020 · 4 comments
Closed

studentify use instructions should be clarified #63

renghen opened this issue Jul 14, 2020 · 4 comments

Comments

@renghen
Copy link

renghen commented Jul 14, 2020

on a fresh clone of the repo lunatech-scala-2-to-scala3-course.

I have tried the following studentify -dot /home/renghen/workspace/dotty/lunatech-scala-2-to-scala3-course /home/renghen/workspace/lunatech-course

But got

YOU HAVE CHANGES IN YOUR GIT WORKSPACE. COMMIT CHANGES AND RE-RUN STUDENTIFY
Executed command: cmt-checkIfWorkspaceClean.sh /home/renghen/workspace/dotty/lunatech-scala-2-to-scala3-course
Working directory: .

@renghen
Copy link
Author

renghen commented Jul 14, 2020

Hi again,

I was able to fix it. In the file core/src/main/scala/com/lightbend/coursegentools/Helpers.scala.

we should add bin/ on line 589 and on line 273.

so on line 273 it becomes "bin/cmt-cpCleanViaGit.sh"
and on line 589 it becomes s"""bin/cmt-checkIfWorkspaceClean.sh ${mainRepo.getPath}"""

@robinraju
Copy link
Contributor

@renghen thanks for reaching out to us.
The Course Management Tools is in active development. This is a known issue and will be fixed in the upcoming release of the tool.

You can also download the studentified artifact from Releases named lunatech-scala-2-to-scala3-course-student.zip

@eloots
Copy link
Contributor

eloots commented Jul 14, 2020

@renghen
To add to what @robinraju said: we will generate binaries for the CMT commands (mainadm, studentify, linearize and delinearize) as part of our CI/CD chain.
These will come with the cmt-cpCleanViaGit.sh and cmt-checkIfWorkspaceClean.shsupporting script.
When installing them, you'll need to adapt your PATH to accommodate them.
This is the fix you can apply when running the commands as you did: add the bin folder in the course management repo to your PATH.

@eloots eloots changed the title studentify does not work studentify use instructions should be clarified Jul 15, 2020
@eloots
Copy link
Contributor

eloots commented Jul 17, 2020

@renghen We just released version 1.0.0 of the Course Management Tools (CMT). You have binaries of the different CMT commands and installation instructions.

@eloots eloots closed this as completed Jul 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants