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.

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

appengine: updating dependency paths and adding documentation #464

Merged
merged 1 commit into from
Aug 26, 2016
Merged

appengine: updating dependency paths and adding documentation #464

merged 1 commit into from
Aug 26, 2016

Conversation

kenshaw
Copy link
Contributor

@kenshaw kenshaw commented Aug 25, 2016

Ran aefix against the gaedatastore and added some documentation to the
appengine folder along with an example for deploying to the traditional
and flexible environments.

@kenshaw
Copy link
Contributor Author

kenshaw commented Aug 25, 2016

Please review and let me know if this can be integrated. Thanks!

@kenshaw
Copy link
Contributor Author

kenshaw commented Aug 26, 2016

I had to I update the Glide dependencies, and also subsequently updated the go_appengine SDK used with travis-ci.

- export PATH=$PATH:$PWD/go_appengine/
- $PWD/go_appengine/goapp get github.com/stretchr/testify/require github.com/pborman/uuid golang.org/x/net/context # FIXME
# - $PWD/go_appengine/goapp get github.com/stretchr/testify/require github.com/pborman/uuid golang.org/x/net/context
Copy link
Member

Choose a reason for hiding this comment

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

You may safely remove this line if it works and runs the tests without it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do.

@dennwc
Copy link
Member

dennwc commented Aug 26, 2016

What is the appengine/appengine binary file attached?

@kenshaw
Copy link
Contributor Author

kenshaw commented Aug 26, 2016

The appengine/appengine binary is attached because I'm an idiot. I will fix this.

@kenshaw
Copy link
Contributor Author

kenshaw commented Aug 26, 2016

I made the needed changes. Let me know if there is any other problems / issues. Again, thanks!

Ran aefix against the gaedatastore and added some documentation to the
appengine folder along with an example for deploying to the traditional
and flexible environments.
@dennwc
Copy link
Member

dennwc commented Aug 26, 2016

LGTM

@dennwc dennwc merged commit 2dcadd3 into cayleygraph:master Aug 26, 2016
@kenshaw kenshaw deleted the gaedatastore-update-appengine-dependencies branch September 4, 2016 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants