{"action":"created","issue":{"url":"https://api.github.com/repos/purcell/package-lint/issues/70","repository_url":"https://api.github.com/repos/purcell/package-lint","labels_url":"https://api.github.com/repos/purcell/package-lint/issues/70/labels{/name}","comments_url":"https://api.github.com/repos/purcell/package-lint/issues/70/comments","events_url":"https://api.github.com/repos/purcell/package-lint/issues/70/events","html_url":"https://github.com/purcell/package-lint/pull/70","id":234010730,"number":70,"title":"Report checkdoc errors too","user":{"login":"DamienCassou","id":217543,"avatar_url":"https://avatars2.githubusercontent.com/u/217543?v=3","gravatar_id":"","url":"https://api.github.com/users/DamienCassou","html_url":"https://github.com/DamienCassou","followers_url":"https://api.github.com/users/DamienCassou/followers","following_url":"https://api.github.com/users/DamienCassou/following{/other_user}","gists_url":"https://api.github.com/users/DamienCassou/gists{/gist_id}","starred_url":"https://api.github.com/users/DamienCassou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DamienCassou/subscriptions","organizations_url":"https://api.github.com/users/DamienCassou/orgs","repos_url":"https://api.github.com/users/DamienCassou/repos","events_url":"https://api.github.com/users/DamienCassou/events{/privacy}","received_events_url":"https://api.github.com/users/DamienCassou/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":14,"created_at":"2017-06-06T20:14:59Z","updated_at":"2017-06-25T03:15:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/purcell/package-lint/pulls/70","html_url":"https://github.com/purcell/package-lint/pull/70","diff_url":"https://github.com/purcell/package-lint/pull/70.diff","patch_url":"https://github.com/purcell/package-lint/pull/70.patch"},"body":"This adds checkdoc to the list of checks. The only problem I found is that `package-lint.el` has checkdoc warnings so this PR won't build. The problematic documentation is:\r\n\r\n```\r\n(defun package-lint--check-version-regexp-list (valid-deps list rx-start rx-end)\r\n  \"Warn about matches of REGEXP when VERSION is not in VALID-DEPS.\r\nLIST is an alist of (VERSION . REGEXP*).\r\nREGEXP is (concat RX-START REGEXP* RX-END) for each REGEXP*.\"\r\n...)\r\n```\r\n\r\ncheckdoc reports that `\"matches\"` should probably be `\"match\"` instead. I don't know what you want to do about that."},"comment":{"url":"https://api.github.com/repos/purcell/package-lint/issues/comments/310879422","html_url":"https://github.com/purcell/package-lint/pull/70#issuecomment-310879422","issue_url":"https://api.github.com/repos/purcell/package-lint/issues/70","id":310879422,"user":{"login":"purcell","id":5636,"avatar_url":"https://avatars2.githubusercontent.com/u/5636?v=3","gravatar_id":"","url":"https://api.github.com/users/purcell","html_url":"https://github.com/purcell","followers_url":"https://api.github.com/users/purcell/followers","following_url":"https://api.github.com/users/purcell/following{/other_user}","gists_url":"https://api.github.com/users/purcell/gists{/gist_id}","starred_url":"https://api.github.com/users/purcell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/purcell/subscriptions","organizations_url":"https://api.github.com/users/purcell/orgs","repos_url":"https://api.github.com/users/purcell/repos","events_url":"https://api.github.com/users/purcell/events{/privacy}","received_events_url":"https://api.github.com/users/purcell/received_events","type":"User","site_admin":false},"created_at":"2017-06-25T03:15:21Z","updated_at":"2017-06-25T03:15:21Z","body":"> any news about this PR?\r\n\r\nIt's currently not applying cleanly. But as noted above, I'm still not 100% that it's reasonable to consider `checkdoc` issues packaging errors. To avoid noise, I think I'd probably be happier if we simply ran \"checkdoc\" and then generated a single warning on the first or last line indicating that checkdoc is unhappy.\r\n"}}
{"actor":{"display_login":"purcell"}}