{"action":"created","issue":{"url":"https://api.github.com/repos/jekyll/jekyll/issues/7448","repository_url":"https://api.github.com/repos/jekyll/jekyll","labels_url":"https://api.github.com/repos/jekyll/jekyll/issues/7448/labels{/name}","comments_url":"https://api.github.com/repos/jekyll/jekyll/issues/7448/comments","events_url":"https://api.github.com/repos/jekyll/jekyll/issues/7448/events","html_url":"https://github.com/jekyll/jekyll/pull/7448","id":395168683,"node_id":"MDExOlB1bGxSZXF1ZXN0MjQxNjg2MDc0","number":7448,"title":"Fix Interpreter warning from Jekyll::Renderer","user":{"login":"mojavelinux","id":79351,"node_id":"MDQ6VXNlcjc5MzUx","avatar_url":"https://avatars2.githubusercontent.com/u/79351?v=4","gravatar_id":"","url":"https://api.github.com/users/mojavelinux","html_url":"https://github.com/mojavelinux","followers_url":"https://api.github.com/users/mojavelinux/followers","following_url":"https://api.github.com/users/mojavelinux/following{/other_user}","gists_url":"https://api.github.com/users/mojavelinux/gists{/gist_id}","starred_url":"https://api.github.com/users/mojavelinux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mojavelinux/subscriptions","organizations_url":"https://api.github.com/users/mojavelinux/orgs","repos_url":"https://api.github.com/users/mojavelinux/repos","events_url":"https://api.github.com/users/mojavelinux/events{/privacy}","received_events_url":"https://api.github.com/users/mojavelinux/received_events","type":"User","site_admin":false},"labels":[{"id":22580614,"node_id":"MDU6TGFiZWwyMjU4MDYxNA==","url":"https://api.github.com/repos/jekyll/jekyll/labels/fix","name":"fix","color":"eb6420","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2019-01-02T07:36:17Z","updated_at":"2019-01-02T07:56:37Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/jekyll/jekyll/pulls/7448","html_url":"https://github.com/jekyll/jekyll/pull/7448","diff_url":"https://github.com/jekyll/jekyll/pull/7448.diff","patch_url":"https://github.com/jekyll/jekyll/pull/7448.patch"},"body":"Change OR operator to a lazy assignment in Jekyll::Renderer#layouts\r\n\r\nThis is a 🐛 bug fix.\r\n\r\n## Summary\r\n\r\nWhen warnings are enabled, Jekyll emits a rather insidious warning message (inidious because of how many times it occurs).\r\n\r\n> warning: instance variable @layouts not initialized\r\n\r\nThis patch fixes the problem.\r\n\r\n## Context\r\n\r\nBased on other methods in the Jekyll::Renderer class, I believe the intention is to use a lazy assignment operator here instead of an OR operation.\r\n\r\nIf that's not the case, then I can update the patch to allow the OR operator to be used without emitting a warning.\r\n\r\n## Semver Changes\r\n\r\nPatch.\r\n"},"comment":{"url":"https://api.github.com/repos/jekyll/jekyll/issues/comments/450803417","html_url":"https://github.com/jekyll/jekyll/pull/7448#issuecomment-450803417","issue_url":"https://api.github.com/repos/jekyll/jekyll/issues/7448","id":450803417,"node_id":"MDEyOklzc3VlQ29tbWVudDQ1MDgwMzQxNw==","user":{"login":"ashmaroli","id":12479464,"node_id":"MDQ6VXNlcjEyNDc5NDY0","avatar_url":"https://avatars0.githubusercontent.com/u/12479464?v=4","gravatar_id":"","url":"https://api.github.com/users/ashmaroli","html_url":"https://github.com/ashmaroli","followers_url":"https://api.github.com/users/ashmaroli/followers","following_url":"https://api.github.com/users/ashmaroli/following{/other_user}","gists_url":"https://api.github.com/users/ashmaroli/gists{/gist_id}","starred_url":"https://api.github.com/users/ashmaroli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ashmaroli/subscriptions","organizations_url":"https://api.github.com/users/ashmaroli/orgs","repos_url":"https://api.github.com/users/ashmaroli/repos","events_url":"https://api.github.com/users/ashmaroli/events{/privacy}","received_events_url":"https://api.github.com/users/ashmaroli/received_events","type":"User","site_admin":false},"created_at":"2019-01-02T07:56:37Z","updated_at":"2019-01-02T07:56:37Z","author_association":"MEMBER","body":"IMHO, there wouldn't be much of an issue because with this memoization, calling `renderer.layouts` is going to return the value of either `@layouts` or `site.layouts` anyway.."}}
{"actor":{"display_login":"ashmaroli"}}