Thanks to visit codestin.com
Credit goes to github.com

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

Heroku button #45

Merged
merged 8 commits into from
Apr 2, 2015
Merged

Heroku button #45

merged 8 commits into from
Apr 2, 2015

Conversation

freeformz
Copy link
Contributor

Add's a Heroku Button so people can experiment w/o having to install anything locally.

puts 'Server started: http://localhost:3000/'
port = ENV['PORT'].nil? ? 3000 : ENV['PORT'].to_i

puts "Server started: http://localhost:#{port}/"
Copy link
Member

Choose a reason for hiding this comment

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

Can you do the same for the go server and make sure it outputs the right port?

@zpao
Copy link
Member

zpao commented Apr 1, 2015

I really like this idea. A couple additional things...

  1. I assume you confirmed that this works but I must ask...
  2. Can you also update the PHP server so it works the same way with PORT=8000 php server.php?

@freeformz
Copy link
Contributor Author

@zpao I did test it on heroku with my fork as best as possible. I'm happy to fix things if it doesn't work though for some reason (or at least try to get others to fix it).

Otherwise I fixed the other issues raised in the PR.

@zpao
Copy link
Member

zpao commented Apr 2, 2015

👍 Thanks!

zpao added a commit that referenced this pull request Apr 2, 2015
@zpao zpao merged commit a91501d into reactjs:master Apr 2, 2015
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