)]}'
{"README.rst":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"fa528fd9bbcc3131818af5d3c72ae2e85670fee2","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"The following cookbooks are dependencies:"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- \u0027etcd\u0027, \u0027~\u003e 5.6\u0027"},{"line_number":43,"context_line":"- \u0027mariadb\u0027, \u0027~\u003e 3.1\u0027"},{"line_number":44,"context_line":"- \u0027memcached\u0027, \u0027~\u003e 6.0\u0027"},{"line_number":45,"context_line":"- \u0027selinux\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9f560f44_5a946444","line":42,"updated":"2020-09-07 12:33:35.000000000","message":"6.0","commit_id":"0fed2ec2b2ef24c970f5edacffe7a7b3abc1074a"}],"libraries/wrappers.rb":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"fa528fd9bbcc3131818af5d3c72ae2e85670fee2","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"  # Wrapper method to allow to easier spec testing"},{"line_number":25,"context_line":"  def role_included?(role)"},{"line_number":26,"context_line":"    node[\u0027roles\u0027].include?(role) # rubocop:disable ChefModernize/NodeRolesInclude"},{"line_number":27,"context_line":"  end"},{"line_number":28,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"9f560f44_5a3e2429","line":26,"updated":"2020-09-07 12:33:35.000000000","message":"The intended replacement seems to be node.role?(role), do you know when that was introduced? Didn\u0027t find a reference in the release notes. Maybe we could add a note to e.g. change when we require Chef \u003e\u003d16?","commit_id":"0fed2ec2b2ef24c970f5edacffe7a7b3abc1074a"},{"author":{"_account_id":21961,"name":"Lance Albertson","email":"lance@osuosl.org","username":"ramereth"},"change_message_id":"5764045070480ffb1e01793c77eb1b037c8a8f40","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"  # Wrapper method to allow to easier spec testing"},{"line_number":25,"context_line":"  def role_included?(role)"},{"line_number":26,"context_line":"    node[\u0027roles\u0027].include?(role) # rubocop:disable ChefModernize/NodeRolesInclude"},{"line_number":27,"context_line":"  end"},{"line_number":28,"context_line":"end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"9f560f44_9279ae83","line":26,"in_reply_to":"9f560f44_5a3e2429","updated":"2020-09-08 17:10:13.000000000","message":"Actually it looks like I properly fixed this in https://review.opendev.org/#/c/731851/. The only issue I ran into was the spec test was failing. I\u0027ll backport the fixes to the chefspec and properly fix this for Stein. I think this has been included for a while, it was just cookstyle that made it clear that this is the preferred way of doing it.","commit_id":"0fed2ec2b2ef24c970f5edacffe7a7b3abc1074a"}]}