Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove dependency on node.js server #1

Merged
merged 4 commits into from
Jul 27, 2015
Merged

Remove dependency on node.js server #1

merged 4 commits into from
Jul 27, 2015

Conversation

ajrkerr
Copy link
Contributor

@ajrkerr ajrkerr commented Jul 27, 2015

This PR does the following:

  • Setups config file that is not tracked by repo for Forecast.io api key
  • Modifies store to use this config file
  • Also removes js/index.js from being tracked (it's a built file so it has a high churn rate and isn't important to the source)

jerridan added a commit that referenced this pull request Jul 27, 2015
@jerridan
Merge pull request #1 from ajrkerr/master
950770d 
Remove dependency on node.js server
@jerridan jerridan merged commit 950770d into jerridan:master Jul 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ajrkerr @jerridan