{"push_id":3026982544,"size":20,"distinct_size":20,"ref":"refs/heads/issue_661_add_geojson_export_type","head":"22cb037c0c63625c9669258598833849b9d1492d","before":"457fb6e12dca9764d36a407e60812d4db054a8f6","commits":[{"sha":"de37e7a6f35e5072993acad1ad2bf2955df53076","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Refactor GeoJson.toLngLatAlts\n- Make it accept Strings instead of Optional<String> to have a nicer signature","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/de37e7a6f35e5072993acad1ad2bf2955df53076"},{"sha":"9fddb824b7f60a55a80217d86a8965a257905806","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Refactor GeoJson.toFeature\n- Separate logic for non-values and pass only the instanceId to make it nicer to use and test","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/9fddb824b7f60a55a80217d86a8965a257905806"},{"sha":"d8a6a49e27d00f71596b0574c6d65f6ad45d0631","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Refactor GeoJson\n- Hide all possible members behind the package","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/d8a6a49e27d00f71596b0574c6d65f6ad45d0631"},{"sha":"c48fa26e3a4ec1c1f6a934f6536a137987bfc93e","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Extract tool to launch GeoJSON transformations from tests","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/c48fa26e3a4ec1c1f6a934f6536a137987bfc93e"},{"sha":"31e25b2ed5e6306e08ab3555b7cfeb73aa64e09f","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Make the ModelBuilder return always the root and provide methods to locate descendant nodes","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/31e25b2ed5e6306e08ab3555b7cfeb73aa64e09f"},{"sha":"8f773955884851de60a5583842a72082d313b817","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Improve and test GeoJson to handle invalid data\n- GEOPOINTS should have 2, 3, or 4 double components\n- GEOTRACES should have at least 2 points\n- GEOSHAPES should have at least 4 points, and the first and last points should be the same","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/8f773955884851de60a5583842a72082d313b817"},{"sha":"4f71fd9ac637fc23acf54623e592742d1e04d552","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Improve and test GeoJson to hide implementation details\n- Make public only the write and the toFeatures methods","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/4f71fd9ac637fc23acf54623e592742d1e04d552"},{"sha":"fc9a10f5e119bf8fd02de64bb16a5ffc70f38694","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Improve GeoJson.toFeatures() semantics and design\n- Simplify code to be less streamy and more easy-to-understand-imperative\n  - Avoid using Optional for exception management (in the case the type and serialized data is incompatible)\n- Extract to static members the point component and point string separators\n- Extract a generic method to parse bound and unbounded Doubles","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/fc9a10f5e119bf8fd02de64bb16a5ffc70f38694"},{"sha":"618fdd6d922217dcf1aa1a87e2ab0db4b49d8675","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Tidy up Model:\n- Unroll one-liners that were only used once\n- Review visibility of members","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/618fdd6d922217dcf1aa1a87e2ab0db4b49d8675"},{"sha":"a53c0969f017755b3ed1780f3c37f85addcd4e7c","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Review and test the GeoJSON.write method","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/a53c0969f017755b3ed1780f3c37f85addcd4e7c"},{"sha":"f29c9e51c75777f2f9b902a87c5f1d1252bdd09a","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Make domain immutable and fix tests where needed\n- ExportConfiguration and OverridableBoolean are now immutable\n- Created a builder for testing","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/f29c9e51c75777f2f9b902a87c5f1d1252bdd09a"},{"sha":"e1a77ffba5cbe52d9860f40842e83a218cdac5e1","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Implement ExportConfiguration invariants to improve validation\n- Also base on DateRange instead of start/end dates","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/e1a77ffba5cbe52d9860f40842e83a218cdac5e1"},{"sha":"bf91830380b487080af383ad1139839168bc83c1","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Simplify ExportConfiguration by replacing ErrorOr with an Optional","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/bf91830380b487080af383ad1139839168bc83c1"},{"sha":"7e3c03778dc8858842897de06518db61fcdb3752","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Conditionally launch a CSV or GeoJSON export from the UI","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/7e3c03778dc8858842897de06518db61fcdb3752"},{"sha":"316ecf9d108b14f056c6196de7fd1d59dcf7924e","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Add new overload to ExportType.run to call it without Analytics","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/316ecf9d108b14f056c6196de7fd1d59dcf7924e"},{"sha":"1e738ff02daf1369326bd90f3193b6d96fc18255","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Add a user friendly factory in ExportType\n- ExportType.valueOf(String) won't deal with lowercase & uppercase combinations in user input","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/1e738ff02daf1369326bd90f3193b6d96fc18255"},{"sha":"558de3666af1feb4d67681124b84390d35613d72","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Declare and use a new CLI param to select the export type (CSV/GeoJSON)\n- Set the new param's default value to CSV\n- Set the legacy launcher (BriefcaseCLI) to always launch CSV exports","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/558de3666af1feb4d67681124b84390d35613d72"},{"sha":"67f781f6031b5dbaefbcef9512c548a79ba07031","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Deal with unsupported ExportTypes in a nicer way\n- Also produce a user-friendly error","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/67f781f6031b5dbaefbcef9512c548a79ba07031"},{"sha":"542e19464dc789879741b378b9d8ec2ecab8450d","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Simplify the awkward creation of ExportTypes\n- By using the factory method that accepts an Optional<Analytics> we can inject one factory only, at the cost of making that factory method more visible (package only), which seems a tradeoff worth paying","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/542e19464dc789879741b378b9d8ec2ecab8450d"},{"sha":"22cb037c0c63625c9669258598833849b9d1492d","author":{"name":"Guille","email":"691aef3809f4b4d7d8455fd7901ba763bfe3c6e5@gmail.com"},"message":"Make the naming of custom arity versions of Function consistent\n- We have decided to follow the Java's naming pattern (-, Bi, Tri,...)\n- This will make awkward an arity of more than 3, which is good to prevent doing it. If you ever need a lambda with more than 3 args, odds are you're better off doing something else.","distinct":true,"url":"https://api.github.com/repos/ggalmazor/briefcase/commits/22cb037c0c63625c9669258598833849b9d1492d"}]}
{"actor":{"display_login":"ggalmazor"}}