{"action":"opened","issue":{"url":"https://api.github.com/repos/jimhester/lintr/issues/47","labels_url":"https://api.github.com/repos/jimhester/lintr/issues/47/labels{/name}","comments_url":"https://api.github.com/repos/jimhester/lintr/issues/47/comments","events_url":"https://api.github.com/repos/jimhester/lintr/issues/47/events","html_url":"https://github.com/jimhester/lintr/issues/47","id":56584230,"number":47,"title":"lint_package('lintr')","user":{"login":"ttriche","id":4386,"avatar_url":"https://avatars.githubusercontent.com/u/4386?v=3","gravatar_id":"","url":"https://api.github.com/users/ttriche","html_url":"https://github.com/ttriche","followers_url":"https://api.github.com/users/ttriche/followers","following_url":"https://api.github.com/users/ttriche/following{/other_user}","gists_url":"https://api.github.com/users/ttriche/gists{/gist_id}","starred_url":"https://api.github.com/users/ttriche/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ttriche/subscriptions","organizations_url":"https://api.github.com/users/ttriche/orgs","repos_url":"https://api.github.com/users/ttriche/repos","events_url":"https://api.github.com/users/ttriche/events{/privacy}","received_events_url":"https://api.github.com/users/ttriche/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-02-04T20:40:53Z","updated_at":"2015-02-04T20:40:53Z","closed_at":null,"body":"So since I haven't the foggiest idea what I'm doing (at the moment I'm trying to enforce camelCase instead of underscores_between_variable_name_bits, blah blah), I figured I'd start by linting lintr.\r\n\r\nlibrary(lintr)\r\nlintrCheckResults <- lint_package('lintr')\r\n## ...time passes...\r\n## There were 50 or more warnings (use warnings() to see the first 50)\r\n\r\nlibrary(dplyr)\r\nwarnings() %>% unique\r\n## Warning messages:\r\n## 1: In source_file$parse : partial match of 'parse' to 'parsed_content'\r\n\r\nlintrCheckResults %>% tail(5) %>% head(2)\r\n## [[1]]\r\n## ./tests/testthat/test-rstudio_markers.R:74:26: style: Variable and function names should be all lowercase.\r\n##    expect_equal(marker3$basePath, \"test\"),\r\n##                         ^~~~~~~~\r\n##\r\n## [[2]]\r\n## ./tests/testthat/test-rstudio_markers.R:76:1: style: lines should not be more than 80 characters.\r\n##    expect_equal(marker3$markers[[1]]$file, file.path(\"test\", lint3[[1]]$filename)),\r\n## ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\r\n## ...\r\n\r\nSo some thoughts (which will turn into a pull request momentarily):\r\n\r\n1) lintr could stand to pass its own linting (modulo bad.R, which should probably be in inst/example)\r\n2) it would be cool if results of linting a package could be organized in a hierarchy, like the package.\r\n\r\n"}}