{"push_id":1314795857,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"0f8d74891e875a89b3fddf621fe7f3c34b08db7b","before":"663d17cb3f7b94a61693c5f78e5257c8feb121eb","commits":[{"sha":"eb1b505201fdff4a517468a53c9ee41b1cc84b9a","author":{"name":"savef","email":"4ba3b79cd198902d4048434805e7dc5521016a05@users.noreply.github.com"},"message":"Fix false positive in `Lint/AssignmentInCondition` (#3520) (#3529)\n\nThis was broken in b0d8e0cf which was simply a refactoring commit. It looks like part of the code was omitted accidentally when porting to the new block. Specifically, the part which was dropped was: https://github.com/bbatsov/rubocop/blob/78de463f33992bce5b1ecf40e311958ad773bad5/lib/rubocop/cop/lint/assignment_in_condition.rb#L35-L38\r\n\r\nI've ported the missing checks to the `#skip_children?` method, and added a test which would've caught the regression.","distinct":true,"url":"https://api.github.com/repos/rrosenblum/rubocop/commits/eb1b505201fdff4a517468a53c9ee41b1cc84b9a"},{"sha":"0f8d74891e875a89b3fddf621fe7f3c34b08db7b","author":{"name":"Tejas Bubane","email":"f80d71c39b02c7b1a72fd8d677d996d8cd7460c2@gmail.com"},"message":"[Fix #3512] Change error message of `Lint/UnneededSplatExpansion` for array (#3526)\n\nIn case of array splat for method parameter, the default message was\r\nambiguous. `Unnecessary splat expansion.` made people just remove the\r\nsplat expansion and pass the array as parameter which is obviously not\r\nthe same as passing splat arguments. Change the message to\r\n`Pass array contents as separate arguments.` to\r\nthe message in such cases to make the intension more clear.","distinct":true,"url":"https://api.github.com/repos/rrosenblum/rubocop/commits/0f8d74891e875a89b3fddf621fe7f3c34b08db7b"}]}
{"actor":{"display_login":"rrosenblum"}}