{"action":"created","issue":{"url":"https://api.github.com/repos/gap-system/gap/issues/1622","repository_url":"https://api.github.com/repos/gap-system/gap","labels_url":"https://api.github.com/repos/gap-system/gap/issues/1622/labels{/name}","comments_url":"https://api.github.com/repos/gap-system/gap/issues/1622/comments","events_url":"https://api.github.com/repos/gap-system/gap/issues/1622/events","html_url":"https://github.com/gap-system/gap/pull/1622","id":252232753,"number":1622,"title":"kernel: rename funcs handling GAP \"global functions\"","user":{"login":"fingolfin","id":241512,"avatar_url":"https://avatars2.githubusercontent.com/u/241512?v=4","gravatar_id":"","url":"https://api.github.com/users/fingolfin","html_url":"https://github.com/fingolfin","followers_url":"https://api.github.com/users/fingolfin/followers","following_url":"https://api.github.com/users/fingolfin/following{/other_user}","gists_url":"https://api.github.com/users/fingolfin/gists{/gist_id}","starred_url":"https://api.github.com/users/fingolfin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fingolfin/subscriptions","organizations_url":"https://api.github.com/users/fingolfin/orgs","repos_url":"https://api.github.com/users/fingolfin/repos","events_url":"https://api.github.com/users/fingolfin/events{/privacy}","received_events_url":"https://api.github.com/users/fingolfin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-08-23T10:22:27Z","updated_at":"2017-09-13T11:10:56Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/gap-system/gap/pulls/1622","html_url":"https://github.com/gap-system/gap/pull/1622","diff_url":"https://github.com/gap-system/gap/pull/1622.diff","patch_url":"https://github.com/gap-system/gap/pull/1622.patch"},"body":"Looking at the C code, I never knew what these functions were about. So I figured it's best to rename the kernel functions to mach their GAP level counterparts.\r\n\r\nAlso replaced some calls to `NewFunctionT` by calls to `NewFunction`, and fixed typos in comments."},"comment":{"url":"https://api.github.com/repos/gap-system/gap/issues/comments/329135478","html_url":"https://github.com/gap-system/gap/pull/1622#issuecomment-329135478","issue_url":"https://api.github.com/repos/gap-system/gap/issues/1622","id":329135478,"user":{"login":"fingolfin","id":241512,"avatar_url":"https://avatars2.githubusercontent.com/u/241512?v=4","gravatar_id":"","url":"https://api.github.com/users/fingolfin","html_url":"https://github.com/fingolfin","followers_url":"https://api.github.com/users/fingolfin/followers","following_url":"https://api.github.com/users/fingolfin/following{/other_user}","gists_url":"https://api.github.com/users/fingolfin/gists{/gist_id}","starred_url":"https://api.github.com/users/fingolfin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fingolfin/subscriptions","organizations_url":"https://api.github.com/users/fingolfin/orgs","repos_url":"https://api.github.com/users/fingolfin/repos","events_url":"https://api.github.com/users/fingolfin/events{/privacy}","received_events_url":"https://api.github.com/users/fingolfin/received_events","type":"User","site_admin":false},"created_at":"2017-09-13T11:10:56Z","updated_at":"2017-09-13T11:10:56Z","author_association":"OWNER","body":"@stevelinton if that means you approve of the PR, could you please review it accordingly (using the GitHub review feature) ? And/or just \"Rebase and merge\" it? Though it would be good to hear whether @markuspf still objects (and if so, I'd love to hear some more details, as otherwise I don't know what to do with that...)"}}
{"actor":{"display_login":"fingolfin"}}