-
Notifications
You must be signed in to change notification settings - Fork 16
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
Requirements change detection fails for -r sub.pip
#2
Comments
you can work around this by using a separate, standalone, file for each environmentt and point out the file to use with settings:
app:
requirements: requirements-live.txt
# paths ending with .pip or .txt ar passed to 'pip install -r',
# paths ending with .py are passed to 'pyhon {} develop'.format(name) |
Could be done by recursive parse, but should probably be too slow since each file would needed to be fetched from server for comparison. One could instead change the -r usage the other way around, like using a master requirements file for live environment in repo root and then for dev/staging use a separate with debug-addons and a reference to the master. |
I have some progress on this in a local tree, now pushed to feature branch: On Mon, May 25, 2015 at 9:14 AM, Jonas Lundberg notifications@github.com
Joar Wandborg |
If a requirements file includes another requirements file, the app.deploy task won't figure that out. It would have to actually parse the contents to know.
The text was updated successfully, but these errors were encountered: