Skip to content
This repository has been archived by the owner on Apr 18, 2018. It is now read-only.

Fix dependencies, add documentation and start script #154

Merged
merged 2 commits into from
Feb 27, 2015

Conversation

sjaensch
Copy link
Contributor

Getting pushmanager up and running was a bit of a challenge even with the documentation we have available inside of Yelp. This branch tries to fix that.

setup_requires=['setuptools'],
install_requires=[
'PyYAML == 3.11',
'SQLAlchemy == 0.9.8',
'lxml == 2.2.4',
'mysql-python == 1.2.5',
'python-daemon == 1.5.2',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

== in setup.py is usually wrong and reserved for requirements.txt

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, I just followed the style of the project.

asottile added a commit that referenced this pull request Feb 27, 2015
Fix dependencies, add documentation and start script
@asottile asottile merged commit 6384204 into Yelp:master Feb 27, 2015
@kkellyy kkellyy mentioned this pull request Mar 11, 2015
kkellyy added a commit that referenced this pull request Mar 11, 2015
NEW FEATURES

 * Add 'my pushes' view & ability to filter by state on pushes (sjaensch, #152)
 * Add drop-down selector to filter pushes by state (sjaensch, #155)

 IMPROVEMENTS

 * Re-activated flake8 hooks (rockdog, #151)
 * Updated requirements & documentation (sjaensch, #154)
 * New start/stop/reload script (sjaensch, #154)
 * Improve pagination implemention on /pushes (sjaensch, #155)
 * Put pushmaster commands in textarea, so they're easier to select (chunkyg, #157)

 BUG FIXES

 * Strip whitespace surrounding whitespace from push branch name (michalczapko, #153)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants