Lots of better names...

Review status: 0 of 11 files reviewed at latest revision, all discussions resolved, some commit checks broke.

lib/react_on_rails/react_on_rails_helper.rb, line 101 at r1 (raw file):

    #      if the JS code throws
    # Any other options are passed to the content tag, including the id.
    def react_component(component_name, options = {})

options is for a plain Hash

renderer_options is for our special object of configuration when rendering

Comments from Reviewable

@mapreal19 Still review.

Coverage remained the same at ?% when pulling e224450 on small-refactorings into ede0ac8 on master.

just a couple of minor things

Reviewed 9 of 15 files at r1, 2 of 4 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

lib/react_on_rails/react_on_rails_helper.rb, line 435 at r2 (raw file):

        result = ReactOnRails::ServerRenderingPool.server_render_js_with_console_logging(js_code, render_options)
      rescue StandardError => err
        # This error came from the renderer

I'd remove this comment. The reader can see we're raising a prerender error anyway

lib/react_on_rails/server_rendering_pool/ruby_embedded_java_script.rb, line 62 at r2 (raw file):

      class << self
        private

we should leave private here?

Comments from Reviewable