{"action":"created","issue":{"url":"https://api.github.com/repos/seegno/jscs-config-seegno/issues/13","labels_url":"https://api.github.com/repos/seegno/jscs-config-seegno/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/seegno/jscs-config-seegno/issues/13/comments","events_url":"https://api.github.com/repos/seegno/jscs-config-seegno/issues/13/events","html_url":"https://github.com/seegno/jscs-config-seegno/issues/13","id":121208528,"number":13,"title":"False positives for requireSqlTemplateInQueryFunction","user":{"login":"rplopes","id":760760,"avatar_url":"https://avatars.githubusercontent.com/u/760760?v=3","gravatar_id":"","url":"https://api.github.com/users/rplopes","html_url":"https://github.com/rplopes","followers_url":"https://api.github.com/users/rplopes/followers","following_url":"https://api.github.com/users/rplopes/following{/other_user}","gists_url":"https://api.github.com/users/rplopes/gists{/gist_id}","starred_url":"https://api.github.com/users/rplopes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rplopes/subscriptions","organizations_url":"https://api.github.com/users/rplopes/orgs","repos_url":"https://api.github.com/users/rplopes/repos","events_url":"https://api.github.com/users/rplopes/events{/privacy}","received_events_url":"https://api.github.com/users/rplopes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/seegno/jscs-config-seegno/labels/discussion","name":"discussion","color":"d4c5f9"},{"url":"https://api.github.com/repos/seegno/jscs-config-seegno/labels/help%20wanted","name":"help wanted","color":"159818"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-12-09T10:40:02Z","updated_at":"2015-12-11T18:23:14Z","closed_at":null,"body":"This rule seems to suggest that for raw SQL queries, we should use the SQL template, like:\r\n\r\n```\r\nsequelize.query(sql`SELECT * FROM users`)\r\n```\r\n\r\nHowever, when we run jscs with `preset: seegno` without disabling this rule, we get errors like:\r\n\r\n```\r\nUse the `sql` tagged template literal for raw queries at X :\r\n    32 |    let query = sql`UPDATE \"Users\" SET X`;\r\n    33 |\r\n    34 |    return sequelize.query(query)\r\n-----------------------------------^\r\n    35 |      .then(function(result) {\r\n    36 |        done(null, result);\r\n```\r\n\r\nBased on my understanding of the rule, this looks like a false positive. Is my assumption correct, or am I misunderstanding something?"},"comment":{"url":"https://api.github.com/repos/seegno/jscs-config-seegno/issues/comments/164009233","html_url":"https://github.com/seegno/jscs-config-seegno/issues/13#issuecomment-164009233","issue_url":"https://api.github.com/repos/seegno/jscs-config-seegno/issues/13","id":164009233,"user":{"login":"ruiquelhas","id":413255,"avatar_url":"https://avatars.githubusercontent.com/u/413255?v=3","gravatar_id":"","url":"https://api.github.com/users/ruiquelhas","html_url":"https://github.com/ruiquelhas","followers_url":"https://api.github.com/users/ruiquelhas/followers","following_url":"https://api.github.com/users/ruiquelhas/following{/other_user}","gists_url":"https://api.github.com/users/ruiquelhas/gists{/gist_id}","starred_url":"https://api.github.com/users/ruiquelhas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ruiquelhas/subscriptions","organizations_url":"https://api.github.com/users/ruiquelhas/orgs","repos_url":"https://api.github.com/users/ruiquelhas/repos","events_url":"https://api.github.com/users/ruiquelhas/events{/privacy}","received_events_url":"https://api.github.com/users/ruiquelhas/received_events","type":"User","site_admin":false},"created_at":"2015-12-11T18:23:14Z","updated_at":"2015-12-11T18:23:14Z","body":"Same issue I described in my previous comment."}}