GitHub PullRequestReviewCommentEvent at 2014-03-13 09:03:41
Repository: rubocop
Actor: geniou
Payload: {"comment":{"url":"https://api.github.com/repos/bbatsov/rubocop/pulls/comments/10556658","id":10556658,"diff_hunk":"@@ -0,0 +1,69 @@\n+# encoding: utf-8\n+\n+module Rubocop\n+  module Cop\n+    module Style\n+      # This cop checks the style of children definitions at classes and\n+      # modules. Basically there are two different styles:\n+      #\n+      # nested - have each child on its own line\n+      #   class Foo\n+      #     class Bar\n+      #     end\n+      #   end\n+      #\n+      # compact - combine definitions as much as possible\n+      #   class Foo::Bar\n+      #   end\n+      #\n+      # The compact style is only forced, for classes / modules with one child.\n+      class ClassAndModuleChildren < Cop\n+        include ConfigurableEnforcedStyle\n+\n+        NESTED_MSG = 'Use nested module/class definitions instead of ' \\\n+          'compact style.'","path":"lib/rubocop/cop/style/class_and_module_children.rb","position":24,"original_position":24,"commit_id":"86765ceeb44b9092ba0ff0397c21854216c450c0","original_commit_id":"86765ceeb44b9092ba0ff0397c21854216c450c0","user":{"login":"geniou","id":832633,"avatar_url":"https://gravatar.com/avatar/c711799b5ad84ac903fddbdae6001e86?d=https%3A%2F%2Fidenticons.github.com%2F914468502334fdf7c7baf01fb3c95e71.png&r=x","gravatar_id":"c711799b5ad84ac903fddbdae6001e86","url":"https://api.github.com/users/geniou","html_url":"https://github.com/geniou","followers_url":"https://api.github.com/users/geniou/followers","following_url":"https://api.github.com/users/geniou/following{/other_user}","gists_url":"https://api.github.com/users/geniou/gists{/gist_id}","starred_url":"https://api.github.com/users/geniou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geniou/subscriptions","organizations_url":"https://api.github.com/users/geniou/orgs","repos_url":"https://api.github.com/users/geniou/repos","events_url":"https://api.github.com/users/geniou/events{/privacy}","received_events_url":"https://api.github.com/users/geniou/received_events","type":"User","site_admin":false},"body":"Do you mean:\r\n```\r\nMSG = 'AA \\'\r\n      'BB'\r\n```\r\ninstead of\r\n```\r\nMSG = 'AA \\'\r\n  'BB'\r\n```\r\n\r\nIts the same in `AlignArray`, `AlignParameters`, `DefWithParentheses`, `NestedTernaryOperator`, `ConfigurableMax`, 'RequireParentheses' - should I fix it there too? If so, I would make a own commit for this.","created_at":"2014-03-13T09:03:41Z","updated_at":"2014-03-13T09:03:41Z","html_url":"https://github.com/bbatsov/rubocop/pull/869#discussion_r10556658","pull_request_url":"https://api.github.com/repos/bbatsov/rubocop/pulls/869","_links":{"self":{"href":"https://api.github.com/repos/bbatsov/rubocop/pulls/comments/10556658"},"html":{"href":"https://github.com/bbatsov/rubocop/pull/869#discussion_r10556658"},"pull_request":{"href":"https://api.github.com/repos/bbatsov/rubocop/pulls/869"}}}}