{"action":"created","issue":{"url":"https://api.github.com/repos/ocaml/dune/issues/950","repository_url":"https://api.github.com/repos/ocaml/dune","labels_url":"https://api.github.com/repos/ocaml/dune/issues/950/labels{/name}","comments_url":"https://api.github.com/repos/ocaml/dune/issues/950/comments","events_url":"https://api.github.com/repos/ocaml/dune/issues/950/events","html_url":"https://github.com/ocaml/dune/pull/950","id":338117269,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk5MTI5ODc0","number":950,"title":"Syntax for naming dependencies","user":{"login":"rgrinberg","id":139003,"node_id":"MDQ6VXNlcjEzOTAwMw==","avatar_url":"https://avatars3.githubusercontent.com/u/139003?v=4","gravatar_id":"","url":"https://api.github.com/users/rgrinberg","html_url":"https://github.com/rgrinberg","followers_url":"https://api.github.com/users/rgrinberg/followers","following_url":"https://api.github.com/users/rgrinberg/following{/other_user}","gists_url":"https://api.github.com/users/rgrinberg/gists{/gist_id}","starred_url":"https://api.github.com/users/rgrinberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rgrinberg/subscriptions","organizations_url":"https://api.github.com/users/rgrinberg/orgs","repos_url":"https://api.github.com/users/rgrinberg/repos","events_url":"https://api.github.com/users/rgrinberg/events{/privacy}","received_events_url":"https://api.github.com/users/rgrinberg/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":22,"created_at":"2018-07-04T03:20:52Z","updated_at":"2018-07-09T11:31:16Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/ocaml/dune/pulls/950","html_url":"https://github.com/ocaml/dune/pull/950","diff_url":"https://github.com/ocaml/dune/pull/950.diff","patch_url":"https://github.com/ocaml/dune/pull/950.patch"},"body":"This PR is WIP but I'd like to make sure we agree on the syntax before I clean things up and implement the rest.\r\n\r\n@diml I've ended up not including a general `List` constructor in `Dep_conf.t` because I think nesting such a constructor would make things awkward. Instead, I only allow `list` as a \"top level\" constructor.\r\n\r\nAlso, I made `(list ..)` available to unnamed dependencies for symmetric even though it's fairly useless there. I believe that if we have `:x (foo bar baz)` and deleting `:x` causing an error is a bit unnatural.\r\n\r\nRefer to the tests for what it currently looks like, as the implementation is still hairy."},"comment":{"url":"https://api.github.com/repos/ocaml/dune/issues/comments/403448312","html_url":"https://github.com/ocaml/dune/pull/950#issuecomment-403448312","issue_url":"https://api.github.com/repos/ocaml/dune/issues/950","id":403448312,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMzQ0ODMxMg==","user":{"login":"diml","id":1251584,"node_id":"MDQ6VXNlcjEyNTE1ODQ=","avatar_url":"https://avatars1.githubusercontent.com/u/1251584?v=4","gravatar_id":"","url":"https://api.github.com/users/diml","html_url":"https://github.com/diml","followers_url":"https://api.github.com/users/diml/followers","following_url":"https://api.github.com/users/diml/following{/other_user}","gists_url":"https://api.github.com/users/diml/gists{/gist_id}","starred_url":"https://api.github.com/users/diml/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/diml/subscriptions","organizations_url":"https://api.github.com/users/diml/orgs","repos_url":"https://api.github.com/users/diml/repos","events_url":"https://api.github.com/users/diml/events{/privacy}","received_events_url":"https://api.github.com/users/diml/received_events","type":"User","site_admin":false},"created_at":"2018-07-09T11:31:16Z","updated_at":"2018-07-09T11:31:16Z","author_association":"MEMBER","body":"Additionally if we get rid of `%{first-dep}` then the following nice property is true: all the variables that are recognized are either global or appear close to their use site. This is because for `%{deps}` and `%{targets}` the names match the field names."}}
{"actor":{"display_login":"diml"}}