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

Make the project contents a little bit more attractive #74

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jgonggrijp
Copy link
Member

This pull request combines a couple of small edits that should help to improve the appeal of our project a little. Most importantly I added a note to the Readme to repeat that anyone who wants to contribute is welcome, ordered the Dependencies by importance, and replaced the suggestiions in setup.py to restart or logoff/login by useful instructions on how to refresh the environment variables.

I still need to check that the command displayed to Windows users is correct.

@jgonggrijp jgonggrijp self-assigned this Mar 8, 2014
@sleepingdrone
Copy link
Contributor

I tested the command on windows and it works, but I don't think it's very user friendly to execute since I can't copy and paste it due to line wrap and the way select and copy work in the Windows commandline.

We're done! Now all you need to do is run these commands:

  for /f "tokens=3* skip=2" %i in ('reg query "HKCU\Environment" /v RED_SPIDER_R
OOT') do (set RED_SPIDER_ROOT=%i)
  rsshell

@jgonggrijp
Copy link
Member Author

Thanks for testing. I seem to recall I also walked into some other issue with that command when I tested it myself. Back to the drawing table!

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