{"action":"created","issue":{"url":"https://api.github.com/repos/coq/coq/issues/6524","repository_url":"https://api.github.com/repos/coq/coq","labels_url":"https://api.github.com/repos/coq/coq/issues/6524/labels{/name}","comments_url":"https://api.github.com/repos/coq/coq/issues/6524/comments","events_url":"https://api.github.com/repos/coq/coq/issues/6524/events","html_url":"https://github.com/coq/coq/pull/6524","id":285153398,"number":6524,"title":"[print] Restrict use of \"debug\" Termops printer.","user":{"login":"ejgallego","id":7192257,"avatar_url":"https://avatars2.githubusercontent.com/u/7192257?v=4","gravatar_id":"","url":"https://api.github.com/users/ejgallego","html_url":"https://github.com/ejgallego","followers_url":"https://api.github.com/users/ejgallego/followers","following_url":"https://api.github.com/users/ejgallego/following{/other_user}","gists_url":"https://api.github.com/users/ejgallego/gists{/gist_id}","starred_url":"https://api.github.com/users/ejgallego/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejgallego/subscriptions","organizations_url":"https://api.github.com/users/ejgallego/orgs","repos_url":"https://api.github.com/users/ejgallego/repos","events_url":"https://api.github.com/users/ejgallego/events{/privacy}","received_events_url":"https://api.github.com/users/ejgallego/received_events","type":"User","site_admin":false},"labels":[{"id":485985791,"url":"https://api.github.com/repos/coq/coq/labels/kind:%20cleanup","name":"kind: cleanup","color":"3366ff","default":false},{"id":572296906,"url":"https://api.github.com/repos/coq/coq/labels/needs:%20discussion","name":"needs: discussion","color":"bfd4f2","default":false},{"id":733571551,"url":"https://api.github.com/repos/coq/coq/labels/needs:%20overlay","name":"needs: overlay","color":"0f1170","default":false},{"id":394846679,"url":"https://api.github.com/repos/coq/coq/labels/needs:%20review","name":"needs: review","color":"f9d0c4","default":false}],"state":"open","locked":false,"assignee":{"login":"ppedrot","id":1202327,"avatar_url":"https://avatars1.githubusercontent.com/u/1202327?v=4","gravatar_id":"","url":"https://api.github.com/users/ppedrot","html_url":"https://github.com/ppedrot","followers_url":"https://api.github.com/users/ppedrot/followers","following_url":"https://api.github.com/users/ppedrot/following{/other_user}","gists_url":"https://api.github.com/users/ppedrot/gists{/gist_id}","starred_url":"https://api.github.com/users/ppedrot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ppedrot/subscriptions","organizations_url":"https://api.github.com/users/ppedrot/orgs","repos_url":"https://api.github.com/users/ppedrot/repos","events_url":"https://api.github.com/users/ppedrot/events{/privacy}","received_events_url":"https://api.github.com/users/ppedrot/received_events","type":"User","site_admin":false},"assignees":[{"login":"ppedrot","id":1202327,"avatar_url":"https://avatars1.githubusercontent.com/u/1202327?v=4","gravatar_id":"","url":"https://api.github.com/users/ppedrot","html_url":"https://github.com/ppedrot","followers_url":"https://api.github.com/users/ppedrot/followers","following_url":"https://api.github.com/users/ppedrot/following{/other_user}","gists_url":"https://api.github.com/users/ppedrot/gists{/gist_id}","starred_url":"https://api.github.com/users/ppedrot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ppedrot/subscriptions","organizations_url":"https://api.github.com/users/ppedrot/orgs","repos_url":"https://api.github.com/users/ppedrot/repos","events_url":"https://api.github.com/users/ppedrot/events{/privacy}","received_events_url":"https://api.github.com/users/ppedrot/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/coq/coq/milestones/7","html_url":"https://github.com/coq/coq/milestone/7","labels_url":"https://api.github.com/repos/coq/coq/milestones/7/labels","id":2647743,"number":7,"title":"8.9+beta1","description":"The date of the feature freeze has not been announced yet.","creator":{"login":"Zimmi48","id":1108325,"avatar_url":"https://avatars2.githubusercontent.com/u/1108325?v=4","gravatar_id":"","url":"https://api.github.com/users/Zimmi48","html_url":"https://github.com/Zimmi48","followers_url":"https://api.github.com/users/Zimmi48/followers","following_url":"https://api.github.com/users/Zimmi48/following{/other_user}","gists_url":"https://api.github.com/users/Zimmi48/gists{/gist_id}","starred_url":"https://api.github.com/users/Zimmi48/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Zimmi48/subscriptions","organizations_url":"https://api.github.com/users/Zimmi48/orgs","repos_url":"https://api.github.com/users/Zimmi48/repos","events_url":"https://api.github.com/users/Zimmi48/events{/privacy}","received_events_url":"https://api.github.com/users/Zimmi48/received_events","type":"User","site_admin":false},"open_issues":135,"closed_issues":63,"state":"open","created_at":"2017-07-18T15:03:25Z","updated_at":"2018-04-24T14:23:29Z","due_on":null,"closed_at":null},"comments":5,"created_at":"2017-12-29T20:48:44Z","updated_at":"2018-04-24T14:26:09Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/coq/coq/pulls/6524","html_url":"https://github.com/coq/coq/pull/6524","diff_url":"https://github.com/coq/coq/pull/6524.diff","patch_url":"https://github.com/coq/coq/pull/6524.patch"},"body":"The `Termops.print_*` functions are declared as a debug printers, but\r\nthey are pervasively used in non-debug code, and in fact, replaced by\r\nthe real printers defined in `printing`.\r\n\r\nThis seems to point to a design problem in the printing system and in\r\nparticular in its location.\r\n\r\nWe mark the users, and switch some from `Termops` to `Printer`.\r\n"},"comment":{"url":"https://api.github.com/repos/coq/coq/issues/comments/383951964","html_url":"https://github.com/coq/coq/pull/6524#issuecomment-383951964","issue_url":"https://api.github.com/repos/coq/coq/issues/6524","id":383951964,"user":{"login":"herbelin","id":460771,"avatar_url":"https://avatars1.githubusercontent.com/u/460771?v=4","gravatar_id":"","url":"https://api.github.com/users/herbelin","html_url":"https://github.com/herbelin","followers_url":"https://api.github.com/users/herbelin/followers","following_url":"https://api.github.com/users/herbelin/following{/other_user}","gists_url":"https://api.github.com/users/herbelin/gists{/gist_id}","starred_url":"https://api.github.com/users/herbelin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/herbelin/subscriptions","organizations_url":"https://api.github.com/users/herbelin/orgs","repos_url":"https://api.github.com/users/herbelin/repos","events_url":"https://api.github.com/users/herbelin/events{/privacy}","received_events_url":"https://api.github.com/users/herbelin/received_events","type":"User","site_admin":false},"created_at":"2018-04-24T14:26:09Z","updated_at":"2018-04-24T14:26:09Z","author_association":"OWNER","body":"> what is the use case for the lower layers to access user-oriented pretty printing?\r\n\r\nAssuming I understood the question, I would say that it is convenient to have user-oriented printer in lower layers for debugging or fast prototyping of a feature (the debugging printers are often awfully verbose)."}}
{"actor":{"display_login":"herbelin"}}