Few minor suggestions

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

lib/react_on_rails/prerender_error.rb, line 6 at r1 (raw file):

module ReactOnRails
  class PrerenderError < ::ReactOnRails::Error
    # TODO: Consider remove providing original `err` as already have access to `self.cause`

do we create an issue so we don't forget and maybe a newcomer could take this?

lib/react_on_rails/version_syntax_converter.rb, line 5 at r1 (raw file):

require_relative "version"

module ReactOnRails

👍

spec/react_on_rails/configuration_spec.rb, line 11 at r1 (raw file):

    let(:using_webpacker) { false }

    after do

I'd put the after block after the before block

spec/react_on_rails/utils_spec.rb, line 87 at r1 (raw file):

        end

        it { expect(subject).to end_with("public/webpack/development/webpack-bundle.js") }

nice, wasn't aware of end_with

spec/react_on_rails/test_helper/webpack_assets_status_checker_spec.rb, line 60 at r1 (raw file):

      context "when using webpacker, a missing server bundle without hash, and manifest is current" do
        let(:webpack_generated_files) { %w[manifest.json server-bundle.js] }

not for this PR but for tests in the future I'd try to stay of using let for setup.

Is a cop in the rubocop-rspec: https://github.com/rubocop-hq/rubocop-rspec/blob/master/lib/rubocop/cop/rspec/let_setup.rb#L31

Comments from Reviewable

Coverage remained the same at ?% when pulling d85d8c5 on fix-test-helper-for-server-bundle into aaef7c0 on master.

broke into #1102 and #1101