{"push_id":786034949,"size":24,"distinct_size":1,"ref":"refs/heads/master","head":"5004532c13cfc0e481724392898cabc3de712a7a","before":"745bdcece01fa1be9311b4e02515f2a9e802b2be","commits":[{"sha":"fb0cf514e97f96fc78f8ca595928fb0b75edf075","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Whitespace","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/fb0cf514e97f96fc78f8ca595928fb0b75edf075"},{"sha":"4037c49c4b15439a59143ceaf4c196eb97ab0ace","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Store sanitised types from layers parameter\n\nThis moves the list of types created by sanitising the layer API\nparameter from clean.layers to clean.types.from_layers. In subsequent\ncommits, types created from address parsing, and the\nyet-to-be-implemented source parameter will also live in the clean.types\nobject.\n\nThis will allow moving logic to set cmd.type out of controllers, and\ninto separate logic that can be a littler smarter. Also, it will no\nlonger require the clean.default_layers_set flag to be passed all around\nlike a nasty global variable.","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/4037c49c4b15439a59143ceaf4c196eb97ab0ace"},{"sha":"ad8db9b8e9aabebda9185fd73f0edc65e061e165","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Remove layer configuration based on address parsing\n\nThis code doesn't seem like it will be triggered very often (due to it\ncomapring space delimited words with comma delimited words from the text\nfield), and also has the potential to cause quite a bit of weird\nbehavior.","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/ad8db9b8e9aabebda9185fd73f0edc65e061e165"},{"sha":"470458317811353db9ce3415617654f0ca653cbe","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Fix inconsistently named imported module","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/470458317811353db9ce3415617654f0ca653cbe"},{"sha":"e42cb4a746becc8fdc643ecc7fc2fd288dc5d0a5","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Remove seemingly unhelpful comment","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/e42cb4a746becc8fdc643ecc7fc2fd288dc5d0a5"},{"sha":"d35544cdf4861bfb79b04c89ee5ff6c8312317c5","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Remove loops and many global checks from sanitiser tests\n\nModifying these sanitiser tests became extremely hard because almost all\nof them were looping over lots of individual test cases, which places\nassumptions about the common behavior of potentialy very different test\ncases, as well as making assertions about huge swaths of output when\nonly a small amount of that output was really under test.\n\nHopefully these changes will make our tests easier to modify, and not\nreally lose any ability to catch bugs.","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/d35544cdf4861bfb79b04c89ee5ff6c8312317c5"},{"sha":"b9152dbe299bf231b5d8847c976090a2a4aff60e","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Separate concerns of address parser\n\nThe address parser currently does two things:\n1.) make some intelligent guesses as to possible admin regions to\nexplicitly search against to improve the quality of results returned\n2.) make some intelligent guesses as to when no part of the query needs\nto search against anything other than admin regions. This somewhat\nimproves the quality of results returned but mostly improves the speed\nof the Elasticsearch query since it's searching significantly fewer\nrecoords.\n\nThese two concerns are now split into two separate methods within the\nquery_parser helper module. They are mostly independent today, but don't\nhave to be in the future.","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/b9152dbe299bf231b5d8847c976090a2a4aff60e"},{"sha":"1573f3a3fac92aa96aab08176afc6d39240494f1","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Move cmd.type setting logic to types helper","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/1573f3a3fac92aa96aab08176afc6d39240494f1"},{"sha":"3ff175243478bc8115251cf49e0aa439a822d03e","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Remove default_layers_set flag!\n\nAfter refactoring, this flag is no longer needed, as all areas of the\ncode that care about layers do so by setting a key within clean.types,\nand then the types helper intelligently combines those together later.","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/3ff175243478bc8115251cf49e0aa439a822d03e"},{"sha":"24349a3839d3712bedecd57dbbbd3a03c34b6eb3","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Rename query/indidces to query/types\n\nIt didn't really contain a list of indices.","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/24349a3839d3712bedecd57dbbbd3a03c34b6eb3"},{"sha":"ef42ad4d2ee8b17ddf1a25af515c651988b98669","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Add sources -> types mapping file","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/ef42ad4d2ee8b17ddf1a25af515c651988b98669"},{"sha":"3479325fa84690b238e2fb40a725ee0268e68282","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Add source parameter sanitiser","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/3479325fa84690b238e2fb40a725ee0268e68282"},{"sha":"eed010835ba9b719e8014690c22fadae0a1f9fb9","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Fix reference to geoname\n\nThis is just in a unit test, so it technically passes, but geonames is\nnot a valid layer option (geoname, singular, is used instead), so it's\nbest to correct.","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/eed010835ba9b719e8014690c22fadae0a1f9fb9"},{"sha":"a479de7527d1633a56880c53d3ddb41c16648c6d","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Calculate intersection of types requested by source and layers params","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/a479de7527d1633a56880c53d3ddb41c16648c6d"},{"sha":"6f0ad678eb91b9c089d71f67bf6a8cf4a1abf39e","author":{"email":"354106e416911844661e1603d99ecf7d1c318dda@simioni.org","name":"Julian Simioni"},"message":"Add types middleware\n\nThis middleware looks at the list of types that will be sent to\nElasticsearch, if it's an empty array, it sends an error response before\nElasticsearch is even quieried, because Elasticsearch interprets an\nempty type array as \"search anything\" rather than the intended \"don't\nsearch anything\".","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/6f0ad678eb91b9c089d71f67bf6a8cf4a1abf39e"},{"sha":"f5e182c63b41cb62d56b6a38d4cb06175abb91fa","author":{"email":"b2719240e190e2a649150d94db50be82838efeb0@rcpt.at","name":"Peter Johnson"},"message":"fix tests, clean by moving fixtures to fixtures dir","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/f5e182c63b41cb62d56b6a38d4cb06175abb91fa"},{"sha":"6f817938bce0e675961738082d36f611236968b7","author":{"email":"b2719240e190e2a649150d94db50be82838efeb0@rcpt.at","name":"Peter Johnson"},"message":"refactor search","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/6f817938bce0e675961738082d36f611236968b7"},{"sha":"34cd25a023d6133fcdff731725b8b201c14eaf13","author":{"email":"b2719240e190e2a649150d94db50be82838efeb0@rcpt.at","name":"Peter Johnson"},"message":"refactor leftovers for clarity","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/34cd25a023d6133fcdff731725b8b201c14eaf13"},{"sha":"dd715392545d81245785a8a5be3df9db71cc8f58","author":{"email":"b2719240e190e2a649150d94db50be82838efeb0@rcpt.at","name":"Peter Johnson"},"message":"add npm dependency","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/dd715392545d81245785a8a5be3df9db71cc8f58"},{"sha":"1f13bafab0ac8d935096a22dabc5484ca99855dc","author":{"email":"b2719240e190e2a649150d94db50be82838efeb0@rcpt.at","name":"Peter Johnson"},"message":"refactor reverse & tests","distinct":false,"url":"https://api.github.com/repos/pelias/api/commits/1f13bafab0ac8d935096a22dabc5484ca99855dc"}]}