{"action":"created","issue":{"url":"https://api.github.com/repos/chapel-lang/pychapel/issues/74","repository_url":"https://api.github.com/repos/chapel-lang/pychapel","labels_url":"https://api.github.com/repos/chapel-lang/pychapel/issues/74/labels{/name}","comments_url":"https://api.github.com/repos/chapel-lang/pychapel/issues/74/comments","events_url":"https://api.github.com/repos/chapel-lang/pychapel/issues/74/events","html_url":"https://github.com/chapel-lang/pychapel/pull/74","id":233499710,"number":74,"title":"docs: Make line-limited literalincludes less fragile","user":{"login":"cassella","id":590808,"avatar_url":"https://avatars1.githubusercontent.com/u/590808?v=3","gravatar_id":"","url":"https://api.github.com/users/cassella","html_url":"https://github.com/cassella","followers_url":"https://api.github.com/users/cassella/followers","following_url":"https://api.github.com/users/cassella/following{/other_user}","gists_url":"https://api.github.com/users/cassella/gists{/gist_id}","starred_url":"https://api.github.com/users/cassella/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cassella/subscriptions","organizations_url":"https://api.github.com/users/cassella/orgs","repos_url":"https://api.github.com/users/cassella/repos","events_url":"https://api.github.com/users/cassella/events{/privacy}","received_events_url":"https://api.github.com/users/cassella/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-06-05T05:47:29Z","updated_at":"2017-06-06T15:55:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chapel-lang/pychapel/pulls/74","html_url":"https://github.com/chapel-lang/pychapel/pull/74","diff_url":"https://github.com/chapel-lang/pychapel/pull/74.diff","patch_url":"https://github.com/chapel-lang/pychapel/pull/74.patch"},"body":"Particularly, harden them against code motion.\r\n\r\nUse :end-before: instead of line numbers to stop the full program\r\nlistings before the test routines.  To this end, add a sentinel\r\ncomment line to the python programs where the literalinclude should\r\nend.\r\n\r\nUse the :pyobject: line-selection helper in the one case it applies --\r\nthe pure python quant().  It can't be used for the Chapel version\r\nbecause it won't include the leading @Chapel().\r\n\r\nI didn't use :begin-after: and :end-before: sentinels for the\r\n@Chapel() quant() because they'd be included in the full-program\r\nlisting.\r\n\r\nTested with a local make doc, and rendering on readthedocs.io."},"comment":{"url":"https://api.github.com/repos/chapel-lang/pychapel/issues/comments/306531699","html_url":"https://github.com/chapel-lang/pychapel/pull/74#issuecomment-306531699","issue_url":"https://api.github.com/repos/chapel-lang/pychapel/issues/74","id":306531699,"user":{"login":"cassella","id":590808,"avatar_url":"https://avatars1.githubusercontent.com/u/590808?v=3","gravatar_id":"","url":"https://api.github.com/users/cassella","html_url":"https://github.com/cassella","followers_url":"https://api.github.com/users/cassella/followers","following_url":"https://api.github.com/users/cassella/following{/other_user}","gists_url":"https://api.github.com/users/cassella/gists{/gist_id}","starred_url":"https://api.github.com/users/cassella/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cassella/subscriptions","organizations_url":"https://api.github.com/users/cassella/orgs","repos_url":"https://api.github.com/users/cassella/repos","events_url":"https://api.github.com/users/cassella/events{/privacy}","received_events_url":"https://api.github.com/users/cassella/received_events","type":"User","site_admin":false},"created_at":"2017-06-06T15:55:02Z","updated_at":"2017-06-06T15:55:02Z","body":"Though on the other hand, since it's start-_after_, the line in the intro comment that's used as the tag won't itself be included in the targeted include.  And it seems like the tag string would need to make it obvious to passers-by that it's being used externally, so please don't rephrase it without changing the docs too...\r\n\r\n\r\nI wonder if the sphinx project would be amenable to having the `pyobject` directive include decorators like `@Chapel()`."}}
{"actor":{"display_login":"cassella"}}