{"action":"created","issue":{"url":"https://api.github.com/repos/nest/nest-simulator/issues/95","labels_url":"https://api.github.com/repos/nest/nest-simulator/issues/95/labels{/name}","comments_url":"https://api.github.com/repos/nest/nest-simulator/issues/95/comments","events_url":"https://api.github.com/repos/nest/nest-simulator/issues/95/events","html_url":"https://github.com/nest/nest-simulator/pull/95","id":102868249,"number":95,"title":"Add a script that checks static analysis from TravisCI locally.","user":{"login":"tammoippen","id":4425925,"avatar_url":"https://avatars.githubusercontent.com/u/4425925?v=3","gravatar_id":"","url":"https://api.github.com/users/tammoippen","html_url":"https://github.com/tammoippen","followers_url":"https://api.github.com/users/tammoippen/followers","following_url":"https://api.github.com/users/tammoippen/following{/other_user}","gists_url":"https://api.github.com/users/tammoippen/gists{/gist_id}","starred_url":"https://api.github.com/users/tammoippen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tammoippen/subscriptions","organizations_url":"https://api.github.com/users/tammoippen/orgs","repos_url":"https://api.github.com/users/tammoippen/repos","events_url":"https://api.github.com/users/tammoippen/events{/privacy}","received_events_url":"https://api.github.com/users/tammoippen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-08-24T19:03:43Z","updated_at":"2015-08-27T14:38:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nest/nest-simulator/pulls/95","html_url":"https://github.com/nest/nest-simulator/pull/95","diff_url":"https://github.com/nest/nest-simulator/pull/95.diff","patch_url":"https://github.com/nest/nest-simulator/pull/95.patch"},"body":"This solves #38.\r\n\r\nIt checks, if the selected executables are working and have the correct version. Then it performs the static code analysis from the TravisCI and gives further information to the user.\r\nIt has an incremental mode, which allows to check one file at a time or the user can specify the file he wants to be checked. The CLI arguments follow the implementation from `testsuite/do_tests.sh`.\r\n\r\nThe documentation can be found in #96 ."},"comment":{"url":"https://api.github.com/repos/nest/nest-simulator/issues/comments/135454868","html_url":"https://github.com/nest/nest-simulator/pull/95#issuecomment-135454868","issue_url":"https://api.github.com/repos/nest/nest-simulator/issues/95","id":135454868,"user":{"login":"jougs","id":2678428,"avatar_url":"https://avatars.githubusercontent.com/u/2678428?v=3","gravatar_id":"","url":"https://api.github.com/users/jougs","html_url":"https://github.com/jougs","followers_url":"https://api.github.com/users/jougs/followers","following_url":"https://api.github.com/users/jougs/following{/other_user}","gists_url":"https://api.github.com/users/jougs/gists{/gist_id}","starred_url":"https://api.github.com/users/jougs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jougs/subscriptions","organizations_url":"https://api.github.com/users/jougs/orgs","repos_url":"https://api.github.com/users/jougs/repos","events_url":"https://api.github.com/users/jougs/events{/privacy}","received_events_url":"https://api.github.com/users/jougs/received_events","type":"User","site_admin":false},"created_at":"2015-08-27T14:38:21Z","updated_at":"2015-08-27T14:38:21Z","body":"@abigailm: Could you please check for proper English in the strings in the script and comment?\r\n\r\n@tammoippen: After running the tool, I still have several comments before liking this:\r\n* `--help` should also give a one-line explanation of what the tool is good for.\r\n* The tool expects version 3.6 of clang-format. Yet, after installing it from my Ubuntu 15.04 repositories as explained in [NEST Coding Style Guidelines#Tooling](https://nest.github.io/nest-simulator/coding_guidelines_c++#tooling), I get an error that it is not there. Supplying `--clang-format=clang-format-3.6` works, but I wonder if the default should then not be rather `clang-format-3.6` instead of `clang-format`. Or both should be checked if that's depending on the source of the installed package\r\n* Do we really need version 1.69 of `cppcheck` exactly? That's not available in the Ubuntu 15.04 repositories (1.67 is). Either relax the requirement, or remove the line saying that it can be `apt-get` installed in #96, as that apparently does not work.\r\n* Running on an unmodified branch yields an output of \"Performing static analysis on:\" but no file names (I assume that's because the delta is empty). That output could be \"Nothing to check\" or something similar instead."}}