{"comment":{"url":"https://api.github.com/repos/dimagi/commcare-hq/comments/16470485","html_url":"https://github.com/dimagi/commcare-hq/commit/cc3b7495243b4ddc39738d0bacb9dde2505de7e8#commitcomment-16470485","id":16470485,"user":{"login":"sravfeyn","id":829142,"avatar_url":"https://avatars.githubusercontent.com/u/829142?v=3","gravatar_id":"","url":"https://api.github.com/users/sravfeyn","html_url":"https://github.com/sravfeyn","followers_url":"https://api.github.com/users/sravfeyn/followers","following_url":"https://api.github.com/users/sravfeyn/following{/other_user}","gists_url":"https://api.github.com/users/sravfeyn/gists{/gist_id}","starred_url":"https://api.github.com/users/sravfeyn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sravfeyn/subscriptions","organizations_url":"https://api.github.com/users/sravfeyn/orgs","repos_url":"https://api.github.com/users/sravfeyn/repos","events_url":"https://api.github.com/users/sravfeyn/events{/privacy}","received_events_url":"https://api.github.com/users/sravfeyn/received_events","type":"User","site_admin":false},"position":17,"line":248,"path":"corehq/apps/userreports/expressions/specs.py","commit_id":"cc3b7495243b4ddc39738d0bacb9dde2505de7e8","created_at":"2016-03-03T11:20:14Z","updated_at":"2016-03-03T11:20:14Z","body":"Cool, agree with 2nd point. Will replace 'variables' with context \r\n\r\nRe. 1st point. What I mean is that the context-variables can itself contain expressions so it feels `_expression` suffix is justified. See example [here](https://github.com/dimagi/commcare-hq/pull/10586/files#diff-0ede03ea138027f30e000f57b582d5deR749)"}}