{"action":"created","issue":{"url":"https://api.github.com/repos/akka/akka-http/issues/2117","repository_url":"https://api.github.com/repos/akka/akka-http","labels_url":"https://api.github.com/repos/akka/akka-http/issues/2117/labels{/name}","comments_url":"https://api.github.com/repos/akka/akka-http/issues/2117/comments","events_url":"https://api.github.com/repos/akka/akka-http/issues/2117/events","html_url":"https://github.com/akka/akka-http/pull/2117","id":341550859,"node_id":"MDExOlB1bGxSZXF1ZXN0MjAxNjYyMTI5","number":2117,"title":"make route concatentation lazy and save some allocations","user":{"login":"swachter","id":540079,"node_id":"MDQ6VXNlcjU0MDA3OQ==","avatar_url":"https://avatars2.githubusercontent.com/u/540079?v=4","gravatar_id":"","url":"https://api.github.com/users/swachter","html_url":"https://github.com/swachter","followers_url":"https://api.github.com/users/swachter/followers","following_url":"https://api.github.com/users/swachter/following{/other_user}","gists_url":"https://api.github.com/users/swachter/gists{/gist_id}","starred_url":"https://api.github.com/users/swachter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/swachter/subscriptions","organizations_url":"https://api.github.com/users/swachter/orgs","repos_url":"https://api.github.com/users/swachter/repos","events_url":"https://api.github.com/users/swachter/events{/privacy}","received_events_url":"https://api.github.com/users/swachter/received_events","type":"User","site_admin":false},"labels":[{"id":436859583,"node_id":"MDU6TGFiZWw0MzY4NTk1ODM=","url":"https://api.github.com/repos/akka/akka-http/labels/tested","name":"tested","color":"d7e102","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":11,"created_at":"2018-07-16T14:48:58Z","updated_at":"2018-07-17T11:38:56Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/akka/akka-http/pulls/2117","html_url":"https://github.com/akka/akka-http/pull/2117","diff_url":"https://github.com/akka/akka-http/pull/2117.diff","patch_url":"https://github.com/akka/akka-http/pull/2117.patch"},"body":"This pull request is a follow up on the proposal [Route concatenation operator should use value class and possibly be by name in its second argument](https://discuss.lightbend.com/t/route-concatenation-operator-should-use-value-class-and-possibly-be-by-name-in-its-second-argument/1540).\r\nIt makes two slight modifications to the `RouteWithConcatenation` class that provides the `~` extension method to routes:\r\n\r\n1. The first modification makes `RouteWithConcatenation` a value class.\r\n1. The second modification makes the argument to the `~` method a by-name argument.\r\n\r\nBoth modifications require binary compatibility exception rules."},"comment":{"url":"https://api.github.com/repos/akka/akka-http/issues/comments/405552132","html_url":"https://github.com/akka/akka-http/pull/2117#issuecomment-405552132","issue_url":"https://api.github.com/repos/akka/akka-http/issues/2117","id":405552132,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNTU1MjEzMg==","user":{"login":"ktoso","id":120979,"node_id":"MDQ6VXNlcjEyMDk3OQ==","avatar_url":"https://avatars1.githubusercontent.com/u/120979?v=4","gravatar_id":"","url":"https://api.github.com/users/ktoso","html_url":"https://github.com/ktoso","followers_url":"https://api.github.com/users/ktoso/followers","following_url":"https://api.github.com/users/ktoso/following{/other_user}","gists_url":"https://api.github.com/users/ktoso/gists{/gist_id}","starred_url":"https://api.github.com/users/ktoso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ktoso/subscriptions","organizations_url":"https://api.github.com/users/ktoso/orgs","repos_url":"https://api.github.com/users/ktoso/repos","events_url":"https://api.github.com/users/ktoso/events{/privacy}","received_events_url":"https://api.github.com/users/ktoso/received_events","type":"User","site_admin":false},"created_at":"2018-07-17T11:38:56Z","updated_at":"2018-07-17T11:38:56Z","author_association":"MEMBER","body":"> @ktoso An alternative syntax could be |. The | symbol reflects that combined routes are alternatives (disregarding rejections that are collected over all alternatives.)\r\n\r\nWe will not add more glyphic operators to this DLS, if anything it could be a named combinator like `concatLazily` or similar."}}
{"actor":{"display_login":"ktoso"}}