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

Made this work with runserver: #5

Merged
merged 2 commits into from
Jan 4, 2016

Conversation

Machiel-B
Copy link

I wanted to use this package with django's default devserver for e2e testing purposes.
When I tried adding an option to keep the server running in the background while doing tasks, I ran into some issues with python subprocesses and STDOUT buffering, but I believe I've got those sorted.

  • listens for specific STDOUT message (by regex or string, 'Quit the server with CONTROL-C' by default)
  • triggers task done but keeps server running in background
  • kills server (+ subprocesses) when grunt is done
  • also kills server on SIGINT
  • added an option to allow verbose only for STDERROR, since runserver is prone to spew all the http requests.

- listens for specific STDOUT message (by regex)
- triggers task done but runs in background
- kills server (+ subprocesses) when grunt is done
- also kilss server on SIGINT
nicolaspanel added a commit that referenced this pull request Jan 4, 2016
Made this work with runserver:
@nicolaspanel nicolaspanel merged commit 7a4de11 into nicolaspanel:master Jan 4, 2016
@nicolaspanel
Copy link
Owner

@Machiel-B : thank you!
It is now available from npm (see grunt-contrib-django@0.3.0)

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.

None yet

2 participants