Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Fixed tests, more comprehensive readme, Heroku Cedar ready #2

Merged
merged 4 commits into from
May 11, 2012

Conversation

bensheldon
Copy link
Contributor

Some additions:
#1 All tests (other than pending) now pass (the problem was with the tests)
#2 Updated the readme to explain how to configure Tropo
#3 Added Procfile and sample.env file for Heroku Cedar Stack (or local Foreman-based development)

... though I admit it's a bit of a rollup.

@yuletide
Copy link
Member

SWEEEET
Thanks man!

On Thursday, May 10, 2012 at 9:08 PM, Ben Sheldon wrote:

Some additions:

#1 All tests (other than pending) now pass (the problem was with the tests)
#2 Updated the readme to explain how to configure Tropo
#3 Added Procfile and sample.env file for Heroku Cedar Stack (or local Foreman-based development)

... though I admit it's a bit of a rollup.

You can merge this Pull Request by running:

git pull https://github.com/bensheldon/textyourcity master-fixes

Or you can view, comment on it, or merge it online at:

https://github.com/codeforamerica/textyourcity/pull/2

-- Commit Summary --

  • Updated README with explanation for setting up Tropo and Heroku.
  • Added Procfile and sample .env file for use with Heroku's Cedar stack or local Foreman development.
  • Added Testing section to Readme.
  • Fixed some tests so that now all of them, (other than pending) now pass.

-- File Changes --

M .gitignore (1)
A Procfile (1)
M README.md (http://README.md) (41)
A sample.env (5)
M spec/controllers/responses_controller_spec.rb (8)
M spec/models/poll_spec.rb (6)

-- Patch Links --

https://github.com/codeforamerica/textyourcity/pull/2.patch
https://github.com/codeforamerica/textyourcity/pull/2.diff


Reply to this email directly or view it on GitHub:
https://github.com/codeforamerica/textyourcity/pull/2

@yuletide
Copy link
Member

We could also add in something about debugging locally with tunnlr, pretty easy to setup and use

On Thursday, May 10, 2012 at 9:08 PM, Ben Sheldon wrote:

Some additions:

#1 All tests (other than pending) now pass (the problem was with the tests)
#2 Updated the readme to explain how to configure Tropo
#3 Added Procfile and sample.env file for Heroku Cedar Stack (or local Foreman-based development)

... though I admit it's a bit of a rollup.

You can merge this Pull Request by running:

git pull https://github.com/bensheldon/textyourcity master-fixes

Or you can view, comment on it, or merge it online at:

https://github.com/codeforamerica/textyourcity/pull/2

-- Commit Summary --

  • Updated README with explanation for setting up Tropo and Heroku.
  • Added Procfile and sample .env file for use with Heroku's Cedar stack or local Foreman development.
  • Added Testing section to Readme.
  • Fixed some tests so that now all of them, (other than pending) now pass.

-- File Changes --

M .gitignore (1)
A Procfile (1)
M README.md (http://README.md) (41)
A sample.env (5)
M spec/controllers/responses_controller_spec.rb (8)
M spec/models/poll_spec.rb (6)

-- Patch Links --

https://github.com/codeforamerica/textyourcity/pull/2.patch
https://github.com/codeforamerica/textyourcity/pull/2.diff


Reply to this email directly or view it on GitHub:
https://github.com/codeforamerica/textyourcity/pull/2

yuletide added a commit that referenced this pull request May 11, 2012
Fixed tests, more comprehensive readme, Heroku Cedar ready
@yuletide yuletide merged commit 195d49f into codeforamerica:master May 11, 2012
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.

2 participants