Merge pull request #3927 from speckins/patch-1

Move shebang to the top of `bin/console` template
rubygems · Sep 4, 2020 · aecc90b · aecc90b
2 parents 0ab446f + 962e669
commit aecc90b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/bundler/lib/bundler/templates/newgem/bin/console.tt b/bundler/lib/bundler/templates/newgem/bin/console.tt
@@ -1,6 +1,5 @@
-# frozen_string_literal: true
-
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require "bundler/setup"
 require "<%= config[:namespaced_path] %>"

diff --git a/bundler/spec/commands/newgem_spec.rb b/bundler/spec/commands/newgem_spec.rb
@@ -376,6 +376,8 @@ def create_temporary_dir(dir)
       expect(bundled_app("#{gem_name}/bin/console")).to exist
       expect(bundled_app("#{gem_name}/bin/setup")).to be_executable
       expect(bundled_app("#{gem_name}/bin/console")).to be_executable
+      expect(bundled_app("#{gem_name}/bin/setup").read).to start_with("#!")
+      expect(bundled_app("#{gem_name}/bin/console").read).to start_with("#!")
     end
 
     it "starts with version 0.1.0" do