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

vagrant up fix - only automatically creates dev env first time run on a workspace. #398

Closed
wants to merge 2 commits into from

Conversation

yarko
Copy link
Contributor

@yarko yarko commented Jul 13, 2013

vagrant:

  • check actual IP;
  • only init machine first time, just as native installations;
  • add .bash_profile to permissions fixup;
  • cleanup excessive newlines in making .bash_profile;

README:

  • include admin & debugging use;
  • info about re-initializing VM & manually running create-dev-env.sh;
  • add django admin & debug_toolbar setting info

  - check actual IP;
  - only init machine first time, just as native installations;
  - add .bash_profile to permissions fixup;
  - cleanup excessive newlines in making .bash_profile;

README:
  - include admin & debugging use;
  - info about re-initializing VM & manually running create-dev-env.sh;
  - add django admin & debug_toolbar setting info
@yarko
Copy link
Contributor Author

yarko commented Jul 13, 2013

Per @singingwolfboy, this is a squash/rebase of #378.

@yarko yarko mentioned this pull request Jul 13, 2013
@yarko
Copy link
Contributor Author

yarko commented Jul 14, 2013

@singingwolfboy - tested w/ the merge from current master; I think this is ready to merge.

@singingwolfboy
Copy link
Contributor

Hey @yarko, sorry I didn't get to this last week -- I was out on vacation. I asked @jtauber to look into this while I was gone, but apparently he didn't. :( Since the PR now has conflicts compared to master, I've rebased again and created #453 -- can you take a look and make sure that it looks good? If so, I'll take another look tomorrow at work, and I'll probably be able to merge it then.

@yarko
Copy link
Contributor Author

yarko commented Jul 22, 2013

closing; moved to #453

@yarko yarko closed this Jul 22, 2013
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
e-kolpakov referenced this pull request in open-craft/edx-platform Mar 31, 2015
…icipation-tests

Tests for export discussion participation command
hachiyanagi-ks added a commit to nttks/edx-platform that referenced this pull request Oct 6, 2015
diegomillan pushed a commit to eduNEXT/edx-platform that referenced this pull request Sep 14, 2016
…bility-fix

Cohorts visibility fix from upstream
xavierchan added a commit to xavierchan/edx-platform-1 that referenced this pull request May 20, 2019
MoisesGSalas pushed a commit to eduNEXT/edx-platform that referenced this pull request Jan 12, 2022
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
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

Successfully merging this pull request may close these issues.

2 participants