{"action":"created","comment":{"url":"https://api.github.com/repos/ocaml/ocaml/pulls/comments/120049272","pull_request_review_id":41979753,"id":120049272,"diff_hunk":"@@ -224,6 +235,32 @@ found in link:INSTALL.adoc#bootstrap[INSTALL.adoc].\n \n ==== Github's CI: Travis and AppVeyor\n \n+The script that is run on Travis continuous integration servers is\n+link:.travis-ci.sh[]; its configuration can be found as\n+a Travis configuration file in link:.travis.yml[].\n+\n+For example, if you want to reproduce the default build on your\n+machine, you can use the configuration values and run command taken from\n+link:.travis.yml[]:\n+\n+----\n+CI_KIND=build XARCH=i386 bash -ex .travis-ci.sh\n+----\n+\n+The scripts support two other kinds of tests (values of the\n+`CI_KIND` variable) which both inspect the patch submitted as part of\n+a pull request. `tests` checks that the testsuite has been modified\n+(hopefully, improved) by the patch, and `changes` checks that the\n+link:Changes[] file has been modified (hopefully to add a new entry).\n+\n+These tests rely on the `$TRAVIS_COMMIT_RANGE` variable which you can\n+set explicitly to reproduce them locally.\n+\n+The `changes` check can be disabled by including \"(no change\n+entry needed)\" in one of your commit messages -- but in general all\n+patches submitted should come with a Change entry; see the guidelines in","path":"HACKING.adoc","position":45,"original_position":45,"commit_id":"628790ec85a5acb69acc7e4c0f1591c02140d2ba","original_commit_id":"628790ec85a5acb69acc7e4c0f1591c02140d2ba","user":{"login":"mshinwell","id":1315488,"avatar_url":"https://avatars3.githubusercontent.com/u/1315488?v=3","gravatar_id":"","url":"https://api.github.com/users/mshinwell","html_url":"https://github.com/mshinwell","followers_url":"https://api.github.com/users/mshinwell/followers","following_url":"https://api.github.com/users/mshinwell/following{/other_user}","gists_url":"https://api.github.com/users/mshinwell/gists{/gist_id}","starred_url":"https://api.github.com/users/mshinwell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mshinwell/subscriptions","organizations_url":"https://api.github.com/users/mshinwell/orgs","repos_url":"https://api.github.com/users/mshinwell/repos","events_url":"https://api.github.com/users/mshinwell/events{/privacy}","received_events_url":"https://api.github.com/users/mshinwell/received_events","type":"User","site_admin":false},"body":"\"Changes\"","created_at":"2017-06-05T06:36:24Z","updated_at":"2017-06-05T06:42:09Z","html_url":"https://github.com/ocaml/ocaml/pull/1190#discussion_r120049272","pull_request_url":"https://api.github.com/repos/ocaml/ocaml/pulls/1190","_links":{"self":{"href":"https://api.github.com/repos/ocaml/ocaml/pulls/comments/120049272"},"html":{"href":"https://github.com/ocaml/ocaml/pull/1190#discussion_r120049272"},"pull_request":{"href":"https://api.github.com/repos/ocaml/ocaml/pulls/1190"}}},"pull_request":{"url":"https://api.github.com/repos/ocaml/ocaml/pulls/1190","id":123880039,"html_url":"https://github.com/ocaml/ocaml/pull/1190","diff_url":"https://github.com/ocaml/ocaml/pull/1190.diff","patch_url":"https://github.com/ocaml/ocaml/pull/1190.patch","issue_url":"https://api.github.com/repos/ocaml/ocaml/issues/1190","number":1190,"state":"open","locked":false,"title":"minor `HACKING.adoc` improvements","user":{"login":"gasche","id":426238,"avatar_url":"https://avatars1.githubusercontent.com/u/426238?v=3","gravatar_id":"","url":"https://api.github.com/users/gasche","html_url":"https://github.com/gasche","followers_url":"https://api.github.com/users/gasche/followers","following_url":"https://api.github.com/users/gasche/following{/other_user}","gists_url":"https://api.github.com/users/gasche/gists{/gist_id}","starred_url":"https://api.github.com/users/gasche/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gasche/subscriptions","organizations_url":"https://api.github.com/users/gasche/orgs","repos_url":"https://api.github.com/users/gasche/repos","events_url":"https://api.github.com/users/gasche/events{/privacy}","received_events_url":"https://api.github.com/users/gasche/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-06-04T14:43:13Z","updated_at":"2017-06-05T06:42:09Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f11207522b12db72c02a7cbd36ecec600ddd001f","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/ocaml/ocaml/pulls/1190/commits","review_comments_url":"https://api.github.com/repos/ocaml/ocaml/pulls/1190/comments","review_comment_url":"https://api.github.com/repos/ocaml/ocaml/pulls/comments{/number}","comments_url":"https://api.github.com/repos/ocaml/ocaml/issues/1190/comments","statuses_url":"https://api.github.com/repos/ocaml/ocaml/statuses/628790ec85a5acb69acc7e4c0f1591c02140d2ba","head":{"label":"gasche:hacking.adoc","ref":"hacking.adoc","sha":"628790ec85a5acb69acc7e4c0f1591c02140d2ba","user":{"login":"gasche","id":426238,"avatar_url":"https://avatars1.githubusercontent.com/u/426238?v=3","gravatar_id":"","url":"https://api.github.com/users/gasche","html_url":"https://github.com/gasche","followers_url":"https://api.github.com/users/gasche/followers","following_url":"https://api.github.com/users/gasche/following{/other_user}","gists_url":"https://api.github.com/users/gasche/gists{/gist_id}","starred_url":"https://api.github.com/users/gasche/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gasche/subscriptions","organizations_url":"https://api.github.com/users/gasche/orgs","repos_url":"https://api.github.com/users/gasche/repos","events_url":"https://api.github.com/users/gasche/events{/privacy}","received_events_url":"https://api.github.com/users/gasche/received_events","type":"User","site_admin":false},"repo":{"id":18431480,"name":"ocaml","full_name":"gasche/ocaml","owner":{"login":"gasche","id":426238,"avatar_url":"https://avatars1.githubusercontent.com/u/426238?v=3","gravatar_id":"","url":"https://api.github.com/users/gasche","html_url":"https://github.com/gasche","followers_url":"https://api.github.com/users/gasche/followers","following_url":"https://api.github.com/users/gasche/following{/other_user}","gists_url":"https://api.github.com/users/gasche/gists{/gist_id}","starred_url":"https://api.github.com/users/gasche/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gasche/subscriptions","organizations_url":"https://api.github.com/users/gasche/orgs","repos_url":"https://api.github.com/users/gasche/repos","events_url":"https://api.github.com/users/gasche/events{/privacy}","received_events_url":"https://api.github.com/users/gasche/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/gasche/ocaml","description":" The core OCaml system: compilers, runtime system, base libraries ","fork":true,"url":"https://api.github.com/repos/gasche/ocaml","forks_url":"https://api.github.com/repos/gasche/ocaml/forks","keys_url":"https://api.github.com/repos/gasche/ocaml/keys{/key_id}","collaborators_url":"https://api.github.com/repos/gasche/ocaml/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/gasche/ocaml/teams","hooks_url":"https://api.github.com/repos/gasche/ocaml/hooks","issue_events_url":"https://api.github.com/repos/gasche/ocaml/issues/events{/number}","events_url":"https://api.github.com/repos/gasche/ocaml/events","assignees_url":"https://api.github.com/repos/gasche/ocaml/assignees{/user}","branches_url":"https://api.github.com/repos/gasche/ocaml/branches{/branch}","tags_url":"https://api.github.com/repos/gasche/ocaml/tags","blobs_url":"https://api.github.com/repos/gasche/ocaml/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/gasche/ocaml/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/gasche/ocaml/git/refs{/sha}","trees_url":"https://api.github.com/repos/gasche/ocaml/git/trees{/sha}","statuses_url":"https://api.github.com/repos/gasche/ocaml/statuses/{sha}","languages_url":"https://api.github.com/repos/gasche/ocaml/languages","stargazers_url":"https://api.github.com/repos/gasche/ocaml/stargazers","contributors_url":"https://api.github.com/repos/gasche/ocaml/contributors","subscribers_url":"https://api.github.com/repos/gasche/ocaml/subscribers","subscription_url":"https://api.github.com/repos/gasche/ocaml/subscription","commits_url":"https://api.github.com/repos/gasche/ocaml/commits{/sha}","git_commits_url":"https://api.github.com/repos/gasche/ocaml/git/commits{/sha}","comments_url":"https://api.github.com/repos/gasche/ocaml/comments{/number}","issue_comment_url":"https://api.github.com/repos/gasche/ocaml/issues/comments{/number}","contents_url":"https://api.github.com/repos/gasche/ocaml/contents/{+path}","compare_url":"https://api.github.com/repos/gasche/ocaml/compare/{base}...{head}","merges_url":"https://api.github.com/repos/gasche/ocaml/merges","archive_url":"https://api.github.com/repos/gasche/ocaml/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/gasche/ocaml/downloads","issues_url":"https://api.github.com/repos/gasche/ocaml/issues{/number}","pulls_url":"https://api.github.com/repos/gasche/ocaml/pulls{/number}","milestones_url":"https://api.github.com/repos/gasche/ocaml/milestones{/number}","notifications_url":"https://api.github.com/repos/gasche/ocaml/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/gasche/ocaml/labels{/name}","releases_url":"https://api.github.com/repos/gasche/ocaml/releases{/id}","deployments_url":"https://api.github.com/repos/gasche/ocaml/deployments","created_at":"2014-04-04T07:49:39Z","updated_at":"2016-12-29T14:16:54Z","pushed_at":"2017-06-04T16:45:52Z","git_url":"git://github.com/gasche/ocaml.git","ssh_url":"git@github.com:gasche/ocaml.git","clone_url":"https://github.com/gasche/ocaml.git","svn_url":"https://github.com/gasche/ocaml","homepage":"","size":161065,"stargazers_count":0,"watchers_count":0,"language":"OCaml","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"trunk"}},"base":{"label":"ocaml:trunk","ref":"trunk","sha":"93c087ae28a322764acc80895d198f626b7d2fea","user":{"login":"ocaml","id":1841483,"avatar_url":"https://avatars0.githubusercontent.com/u/1841483?v=3","gravatar_id":"","url":"https://api.github.com/users/ocaml","html_url":"https://github.com/ocaml","followers_url":"https://api.github.com/users/ocaml/followers","following_url":"https://api.github.com/users/ocaml/following{/other_user}","gists_url":"https://api.github.com/users/ocaml/gists{/gist_id}","starred_url":"https://api.github.com/users/ocaml/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ocaml/subscriptions","organizations_url":"https://api.github.com/users/ocaml/orgs","repos_url":"https://api.github.com/users/ocaml/repos","events_url":"https://api.github.com/users/ocaml/events{/privacy}","received_events_url":"https://api.github.com/users/ocaml/received_events","type":"Organization","site_admin":false},"repo":{"id":6786166,"name":"ocaml","full_name":"ocaml/ocaml","owner":{"login":"ocaml","id":1841483,"avatar_url":"https://avatars0.githubusercontent.com/u/1841483?v=3","gravatar_id":"","url":"https://api.github.com/users/ocaml","html_url":"https://github.com/ocaml","followers_url":"https://api.github.com/users/ocaml/followers","following_url":"https://api.github.com/users/ocaml/following{/other_user}","gists_url":"https://api.github.com/users/ocaml/gists{/gist_id}","starred_url":"https://api.github.com/users/ocaml/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ocaml/subscriptions","organizations_url":"https://api.github.com/users/ocaml/orgs","repos_url":"https://api.github.com/users/ocaml/repos","events_url":"https://api.github.com/users/ocaml/events{/privacy}","received_events_url":"https://api.github.com/users/ocaml/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/ocaml/ocaml","description":"The core OCaml system: compilers, runtime system, base libraries","fork":false,"url":"https://api.github.com/repos/ocaml/ocaml","forks_url":"https://api.github.com/repos/ocaml/ocaml/forks","keys_url":"https://api.github.com/repos/ocaml/ocaml/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ocaml/ocaml/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ocaml/ocaml/teams","hooks_url":"https://api.github.com/repos/ocaml/ocaml/hooks","issue_events_url":"https://api.github.com/repos/ocaml/ocaml/issues/events{/number}","events_url":"https://api.github.com/repos/ocaml/ocaml/events","assignees_url":"https://api.github.com/repos/ocaml/ocaml/assignees{/user}","branches_url":"https://api.github.com/repos/ocaml/ocaml/branches{/branch}","tags_url":"https://api.github.com/repos/ocaml/ocaml/tags","blobs_url":"https://api.github.com/repos/ocaml/ocaml/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ocaml/ocaml/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ocaml/ocaml/git/refs{/sha}","trees_url":"https://api.github.com/repos/ocaml/ocaml/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ocaml/ocaml/statuses/{sha}","languages_url":"https://api.github.com/repos/ocaml/ocaml/languages","stargazers_url":"https://api.github.com/repos/ocaml/ocaml/stargazers","contributors_url":"https://api.github.com/repos/ocaml/ocaml/contributors","subscribers_url":"https://api.github.com/repos/ocaml/ocaml/subscribers","subscription_url":"https://api.github.com/repos/ocaml/ocaml/subscription","commits_url":"https://api.github.com/repos/ocaml/ocaml/commits{/sha}","git_commits_url":"https://api.github.com/repos/ocaml/ocaml/git/commits{/sha}","comments_url":"https://api.github.com/repos/ocaml/ocaml/comments{/number}","issue_comment_url":"https://api.github.com/repos/ocaml/ocaml/issues/comments{/number}","contents_url":"https://api.github.com/repos/ocaml/ocaml/contents/{+path}","compare_url":"https://api.github.com/repos/ocaml/ocaml/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ocaml/ocaml/merges","archive_url":"https://api.github.com/repos/ocaml/ocaml/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ocaml/ocaml/downloads","issues_url":"https://api.github.com/repos/ocaml/ocaml/issues{/number}","pulls_url":"https://api.github.com/repos/ocaml/ocaml/pulls{/number}","milestones_url":"https://api.github.com/repos/ocaml/ocaml/milestones{/number}","notifications_url":"https://api.github.com/repos/ocaml/ocaml/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ocaml/ocaml/labels{/name}","releases_url":"https://api.github.com/repos/ocaml/ocaml/releases{/id}","deployments_url":"https://api.github.com/repos/ocaml/ocaml/deployments","created_at":"2012-11-20T22:18:22Z","updated_at":"2017-06-05T01:32:21Z","pushed_at":"2017-06-04T16:45:54Z","git_url":"git://github.com/ocaml/ocaml.git","ssh_url":"git@github.com:ocaml/ocaml.git","clone_url":"https://github.com/ocaml/ocaml.git","svn_url":"https://github.com/ocaml/ocaml","homepage":"","size":143266,"stargazers_count":1296,"watchers_count":1296,"language":"OCaml","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":359,"mirror_url":null,"open_issues_count":132,"forks":359,"open_issues":132,"watchers":1296,"default_branch":"trunk"}},"_links":{"self":{"href":"https://api.github.com/repos/ocaml/ocaml/pulls/1190"},"html":{"href":"https://github.com/ocaml/ocaml/pull/1190"},"issue":{"href":"https://api.github.com/repos/ocaml/ocaml/issues/1190"},"comments":{"href":"https://api.github.com/repos/ocaml/ocaml/issues/1190/comments"},"review_comments":{"href":"https://api.github.com/repos/ocaml/ocaml/pulls/1190/comments"},"review_comment":{"href":"https://api.github.com/repos/ocaml/ocaml/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/ocaml/ocaml/pulls/1190/commits"},"statuses":{"href":"https://api.github.com/repos/ocaml/ocaml/statuses/628790ec85a5acb69acc7e4c0f1591c02140d2ba"}}}}
{"actor":{"display_login":"mshinwell"}}