search

chartpath / ramses-elasticsearch

Guest post for Elastic.co
https://www.elastic.co/blog
13 stars 2 forks source link

Technical review #2

Closed ycombinator closed 8 years ago

ycombinator commented 8 years ago

This looks great. I followed all the instructions and they worked, just as expected. Just a couple of minor nitpicks in this PR; feel free to ignore :)

chartpath commented 8 years ago

Thanks!