search

unipop-graph / unipop

Data Integration Graph
Apache License 2.0
203 stars 35 forks source link

Feature/add jest support #41

Closed GurRonen closed 8 years ago

GurRonen commented 8 years ago

Finished feature, works for both elastic 1 and 2.

a few remaining issues before merge:

  1. should the elastic2 module be deleted
  2. process tests fail due to messup in indexing, with pre-existing index and proper configuration
GurRonen commented 8 years ago

implements #35

rmagen commented 8 years ago

Make sure to go over all the comments and see that they're all acknowledged with a fix, counter argument, etc. Thanks :)

GurRonen commented 8 years ago

@rmagen all comments should be completed.

rmagen commented 8 years ago

Great! Some tests aren't passing on this branch. Do you want help fixing them?

rmagen commented 8 years ago

The Jest code has been merged to master from a different branch.