Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

additional fixes for the runner #8

Merged
merged 1 commit into from Jun 28, 2018
Merged

additional fixes for the runner #8

merged 1 commit into from Jun 28, 2018

Conversation

nickfun
Copy link
Contributor

@nickfun nickfun commented Jun 18, 2018

I forgot ~/workspace in the path, so it didn't write the file correctly.

  • Add a SUCCESS output to make install
  • make runner will correctly create the "Bootcamp" runner now

@nickfun nickfun requested a review from jingming June 18, 2018 21:38
@nickfun
Copy link
Contributor Author

nickfun commented Jun 18, 2018

screen shot 2018-06-18 at 2 38 35 pm

Students can hit Run, then choose Bootcamp in the Runner option.

@@ -12,9 +12,14 @@ serve:
. venv/bin/activate; python run.py

install: venv dependencies configure runner
@echo '@@@ INSTALL SUCCESS @@@'
Copy link
Contributor

Choose a reason for hiding this comment

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

LOL

@nickfun nickfun merged commit c762b5d into master Jun 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants