-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Adding a Makefile #415
Adding a Makefile #415
Conversation
Gonna use this to push out v1.8.3 tomorrow morning, let's see what kind of disaster that causes. |
piprot: requirements | ||
piprot -x $(REQUIREMENTS_FILE) | ||
|
||
$(REQUIREMENTS_OUT): $(REQUIREMENTS_FILE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<3 this one weird trick
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't follow... looking for a $(REQUIREMENTS_FILE) as a target and not seeing one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've been using this for other projects for some years now, it's very handy. Basically it only tries to install new requirements if the requirements.txt file is newer than the requirements.txt.log file.
Sadly making Sphinx a requirement makes travisci in Py3 (~4min) run way slower than Py2 (~25s). |
Merged in f6a0e96. |
Partly stolen from #317. (Thanks @kevinburke! Thoughts?)
Was considering doing separate
dev-requirements.txt
(sphinx, piprot, etc) andtest-requirements.txt
(nose, tox, etc) but... so many requirements. :(Then again, we want travisci and co to fail when sphinx build fails (maybe also when piprot says things are out of date?) #414, so merging these things might be a Good Thing.
The plan is to add my release routine in here, too.