{"action":"created","issue":{"url":"https://api.github.com/repos/metacloud/molecule/issues/182","repository_url":"https://api.github.com/repos/metacloud/molecule","labels_url":"https://api.github.com/repos/metacloud/molecule/issues/182/labels{/name}","comments_url":"https://api.github.com/repos/metacloud/molecule/issues/182/comments","events_url":"https://api.github.com/repos/metacloud/molecule/issues/182/events","html_url":"https://github.com/metacloud/molecule/issues/182","id":160003548,"number":182,"title":"linting/validators","user":{"login":"dhutty","id":102618,"avatar_url":"https://avatars.githubusercontent.com/u/102618?v=3","gravatar_id":"","url":"https://api.github.com/users/dhutty","html_url":"https://github.com/dhutty","followers_url":"https://api.github.com/users/dhutty/followers","following_url":"https://api.github.com/users/dhutty/following{/other_user}","gists_url":"https://api.github.com/users/dhutty/gists{/gist_id}","starred_url":"https://api.github.com/users/dhutty/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dhutty/subscriptions","organizations_url":"https://api.github.com/users/dhutty/orgs","repos_url":"https://api.github.com/users/dhutty/repos","events_url":"https://api.github.com/users/dhutty/events{/privacy}","received_events_url":"https://api.github.com/users/dhutty/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-06-13T17:30:05Z","updated_at":"2016-06-14T07:02:19Z","closed_at":null,"body":"There has been much discussion and some code concerning validators/linting with divergent opinions, I'm opening this issue as a place to collect opinions about the direction we want to take in the area of validation. Please add your thoughts below and whether you will be able to dedicate time to developing.\r\n\r\nIt seems that the most sensible compromise would be to support molecule options such that the user's molecule config would contain a list of validators (for which molecule would provide sensible defaults such as those we have now) and allow the command line invocation of molecule to override.\r\n\r\nIt would be extra nice if the invocation could also support severity signalling (see http://search.cpan.org/~thaljef/Perl-Critic-1.126/bin/perlcritic for an example) with various levels such as:\r\n`--validation=False|off|none`\r\n`--validation=basic`\r\n`--validation=all`\r\n\r\nfor which each validator's config in the molecule.yml would have an interpretation.\r\n\r\nThe obvious initial candidates for initially supported validators are:\r\n- ansible-lint for playbooks\r\n- yapf/flake8 for python files\r\n- rubocop for (serverspec) ruby files"},"comment":{"url":"https://api.github.com/repos/metacloud/molecule/issues/comments/225796332","html_url":"https://github.com/metacloud/molecule/issues/182#issuecomment-225796332","issue_url":"https://api.github.com/repos/metacloud/molecule/issues/182","id":225796332,"user":{"login":"kireledan","id":1918646,"avatar_url":"https://avatars.githubusercontent.com/u/1918646?v=3","gravatar_id":"","url":"https://api.github.com/users/kireledan","html_url":"https://github.com/kireledan","followers_url":"https://api.github.com/users/kireledan/followers","following_url":"https://api.github.com/users/kireledan/following{/other_user}","gists_url":"https://api.github.com/users/kireledan/gists{/gist_id}","starred_url":"https://api.github.com/users/kireledan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kireledan/subscriptions","organizations_url":"https://api.github.com/users/kireledan/orgs","repos_url":"https://api.github.com/users/kireledan/repos","events_url":"https://api.github.com/users/kireledan/events{/privacy}","received_events_url":"https://api.github.com/users/kireledan/received_events","type":"User","site_admin":false},"created_at":"2016-06-14T07:02:19Z","updated_at":"2016-06-14T07:02:19Z","body":"To me this seems a bit unnecessary. The only validator that I think molecule would need would be a dry-run or just a quick syntax check. "}}
{"actor":{"display_login":"kireledan"}}