padrino-gen 0.16.0.pre4 → 0.16.0

data/lib/padrino-gen/generators/helper.rb

@@ -4,11 +4,10 @@ module Padrino
     # Responsible for generating route helpers and associated tests within a Padrino application.
     #
     class Helper < Thor::Group
-
       Padrino::Generators.add_generator(:helper, self)
 
-      def self.source_root; File.expand_path(File.dirname(__FILE__)); end
-      def self.banner; "padrino-gen helper [name]"; end
+      def self.source_root; __dir__; end
+      def self.banner; 'padrino-gen helper [name]'; end
 
       include Thor::Actions
       include Padrino::Generators::Actions
@@ -16,12 +15,12 @@ module Padrino
 
       desc "Description:\n\n\tpadrino-gen helper generates a new Padrino helper"
 
-      argument     :name,       :desc => 'The name of your padrino helper'
-      argument     :fields,     :desc => 'The fields for the helper',                            :default => [],     :type => :array
-      class_option :root,       :desc => 'The root destination',                   :aliases => '-r', :default => '.',    :type => :string
-      class_option :app,        :desc => 'The application destination path',       :aliases => '-a', :default => '/app', :type => :string
-      class_option :destroy,                                                       :aliases => '-d', :default => false,  :type => :boolean
-      class_option :namespace,  :desc => 'The name space of your padrino project', :aliases => '-n', :default => '',     :type => :string
+      argument     :name,      desc: 'The name of your padrino helper'
+      argument     :fields,    desc: 'The fields for the helper',                             default: [],     type: :array
+      class_option :root,      desc: 'The root destination',                   aliases: '-r', default: '.',    type: :string
+      class_option :app,       desc: 'The application destination path',       aliases: '-a', default: '/app', type: :string
+      class_option :destroy,                                                   aliases: '-d', default: false,  type: :boolean
+      class_option :namespace, desc: 'The name space of your padrino project', aliases: '-n', default: '',     type: :string
 
       # Show help if no ARGV given
       require_arguments!

data/lib/padrino-gen/generators/mailer.rb

@@ -4,10 +4,9 @@ module Padrino
     # Responsible for the generating mailers and message definitions.
     #
     class Mailer < Thor::Group
-
       Padrino::Generators.add_generator(:mailer, self)
 
-      def self.source_root; File.expand_path(File.dirname(__FILE__)); end
+      def self.source_root; __dir__; end
       def self.banner; 'padrino-gen mailer [name]'; end
 
       include Thor::Actions
@@ -16,12 +15,12 @@ module Padrino
 
       desc "Description:\n\n\tpadrino-gen mailer generates a new Padrino mailer"
 
-      argument     :name,      :desc => 'The name of your padrino mailer'
-      argument     :actions,   :desc => 'The delivery actions to add to your mailer',                                   :type =>  :array, :default =>  []
-      class_option :root,      :desc => 'The root destination',                   :aliases => '-r', :default => '.',    :type => :string
-      class_option :app,       :desc => 'The application destination path',       :aliases => '-a', :default => '/app', :type => :string
-      class_option :destroy,                                                      :aliases => '-d', :default => false,  :type => :boolean
-      class_option :namespace, :desc => 'The name space of your padrino project', :aliases => '-n', :default => '',     :type => :string
+      argument     :name,      desc: 'The name of your padrino mailer'
+      argument     :actions,   desc: 'The delivery actions to add to your mailer',                default: [],     type: :array
+      class_option :root,      desc: 'The root destination',                       aliases: '-r', default: '.',    type: :string
+      class_option :app,       desc: 'The application destination path',           aliases: '-a', default: '/app', type: :string
+      class_option :destroy,                                                       aliases: '-d', default: false,  type: :boolean
+      class_option :namespace, desc: 'The name space of your padrino project',     aliases: '-n', default: '',     type: :string
 
       # Show help if no ARGV given.
       require_arguments!
@@ -42,7 +41,7 @@ module Padrino
           @actions         = actions.map(&:to_sym)
           @short_name      = name.to_s.gsub(/_mailer/i, '').underscore.downcase
           @mailer_basename = @short_name.underscore
-          template "templates/mailer.rb.tt", destination_root(app, 'mailers', "#{@mailer_basename}.rb")
+          template 'templates/mailer.rb.tt', destination_root(app, 'mailers', "#{@mailer_basename}.rb")
           empty_directory destination_root(app, 'views', 'mailers', @mailer_basename)
         else
           say 'You are not at the root of a Padrino application! (config/boot.rb not found)'

data/lib/padrino-gen/generators/migration.rb

@@ -6,8 +6,8 @@ module Padrino
     class Migration < Thor::Group
       Padrino::Generators.add_generator(:migration, self)
 
-      def self.source_root; File.expand_path(File.dirname(__FILE__)); end
-      def self.banner; "padrino-gen migration [name] [fields]"; end
+      def self.source_root; __dir__; end
+      def self.banner; 'padrino-gen migration [name] [fields]'; end
 
       include Thor::Actions
       include Padrino::Generators::Actions
@@ -15,10 +15,10 @@ module Padrino
 
       desc "Description:\n\n\tpadrino-gen migration generates a new migration file"
 
-      argument :name, :desc => 'The name of your padrino migration'
-      argument :columns, :desc => 'The columns for the migration', :type => :array, :default => []
-      class_option :root, :desc => 'The root destination', :aliases => '-r', :default => '.', :type => :string
-      class_option :destroy, :aliases => '-d', :default => false, :type => :boolean
+      argument :name, desc: 'The name of your padrino migration'
+      argument :columns, desc: 'The columns for the migration', type: :array, default: []
+      class_option :root, desc: 'The root destination', aliases: '-r', default: '.', type: :string
+      class_option :destroy, aliases: '-d', default: false, type: :boolean
 
       # Show help if no ARGV given.
       require_arguments!

data/lib/padrino-gen/generators/model.rb

@@ -6,8 +6,8 @@ module Padrino
     class Model < Thor::Group
       Padrino::Generators.add_generator(:model, self)
 
-      def self.source_root; File.expand_path(File.dirname(__FILE__)); end
-      def self.banner; "padrino-gen model [name] [fields]"; end
+      def self.source_root; __dir__; end
+      def self.banner; 'padrino-gen model [name] [fields]'; end
 
       include Thor::Actions
       include Padrino::Generators::Actions
@@ -15,13 +15,13 @@ module Padrino
 
       desc "Description:\n\n\tpadrino-gen model generates a new model and migration files"
 
-      argument :name,               :desc => 'The name of your padrino model'
-      argument :fields,             :desc => 'The fields for the model',                                     :default => [],    :type => :array
-      class_option :root,           :desc => 'The root destination',                       :aliases => '-r', :default => '.',   :type => :string
-      class_option :app,            :desc => 'The application destination path',           :aliases => '-a', :default => '.',   :type => :string
-      class_option :destroy,                                                               :aliases => '-d', :default => false, :type => :boolean
-      class_option :skip_migration,                                                        :aliases => '-s', :default => false, :type => :boolean
-      class_option :force,          :desc => 'Generate model files if app already exists', :aliases => '-f', :default => false, :type => :boolean
+      argument :name,               desc: 'The name of your padrino model'
+      argument :fields,             desc: 'The fields for the model',                                  default: [],    type: :array
+      class_option :root,           desc: 'The root destination',                       aliases: '-r', default: '.',   type: :string
+      class_option :app,            desc: 'The application destination path',           aliases: '-a', default: '.',   type: :string
+      class_option :destroy,                                                            aliases: '-d', default: false, type: :boolean
+      class_option :skip_migration,                                                     aliases: '-s', default: false, type: :boolean
+      class_option :force,          desc: 'Generate model files if app already exists', aliases: '-f', default: false, type: :boolean
 
       # Show help if no ARGV given.
       require_arguments!
@@ -35,7 +35,7 @@ module Padrino
         include_component_module_for(:test)
         migration_name = "create_#{name.pluralize.underscore}"
         apply_default_fields fields
-        create_model_file(name, :fields => fields, :app => app)
+        create_model_file(name, fields: fields, app: app)
         generate_model_test(name) if test?
         create_model_migration(migration_name, name, fields) unless options[:skip_migration]
       end
@@ -51,16 +51,15 @@ module Padrino
         return false unless correct_path?
 
         check_app_existence(app)
-
-        if options[:destroy]
-          self.behavior = :revoke
-        else
-          unless options[:force]
+        if model_name_already_exists?
+          if options[:destroy]
+            self.behavior = :revoke
+          elsif !options[:force]
             say "#{@camel_name} already exists."
-            say "Please, change the name."
+            say 'Please, change the name.'
             return false
           end
-        end if model_name_already_exists?
+        end
 
         return false if has_invalid_fields?
 
@@ -73,7 +72,7 @@ module Padrino
       def model_name_already_exists?
         @camel_name = name.to_s.underscore.camelize
 
-        @project_name = ""
+        @project_name = ''
         @project_name = fetch_project_name
 
         return false unless already_exists?(@camel_name, @project_name)
@@ -86,7 +85,7 @@ module Padrino
       def check_orm
         return true if include_component_module_for(:orm)
 
-        say "<= You need an ORM adapter for run this generator. Sorry!"
+        say '<= You need an ORM adapter for run this generator. Sorry!'
         raise SystemExit
       end
 
@@ -103,9 +102,9 @@ module Padrino
       # Check if the fields are valid
       #
       def has_invalid_fields?
-        if invalids = invalid_fields(fields)
+        if (invalids = invalid_fields(fields))
           say 'Invalid field name:', :red
-          say " #{invalids.join(", ")}"
+          say " #{invalids.join(', ')}"
         end
       end
     end

data/lib/padrino-gen/generators/plugin.rb

@@ -11,8 +11,8 @@ module Padrino
       PLUGIN_URL = 'https://github.com/padrino/padrino-recipes/tree/master/plugins'
       Padrino::Generators.add_generator(:plugin, self)
 
-      def self.source_root; File.expand_path(File.dirname(__FILE__)); end
-      def self.banner; "padrino-gen plugin [plugin_identifier] [options]"; end
+      def self.source_root; __dir__; end
+      def self.banner; 'padrino-gen plugin [plugin_identifier] [options]'; end
 
       include Thor::Actions
       include Padrino::Generators::Actions
@@ -20,11 +20,11 @@ module Padrino
 
       desc "Description:\n\n\tpadrino-gen plugin sets up a plugin within a Padrino application"
 
-      argument :plugin_file, :desc => "The name or path to the Padrino plugin", :optional => true
+      argument :plugin_file, desc: 'The name or path to the Padrino plugin', optional: true
 
-      class_option :root, :desc => "The root destination", :aliases => '-r', :default => ".",   :type => :string
-      class_option :list, :desc => "list available plugins", :aliases => '-l', :default => false, :type => :boolean
-      class_option :destroy, :aliases => '-d', :default => false, :type => :boolean
+      class_option :root, desc: 'The root destination', aliases: '-r', default: '.',   type: :string
+      class_option :list, desc: 'list available plugins', aliases: '-l', default: false, type: :boolean
+      class_option :destroy, aliases: '-d', default: false, type: :boolean
 
       # Show help if no ARGV given.
       require_arguments!
@@ -41,7 +41,7 @@ module Padrino
             self.behavior = :revoke if options[:destroy]
             execute_runner(:plugin, plugin_file)
           else
-            say "You are not at the root of a Padrino application! (config/boot.rb not found)"
+            say 'You are not at the root of a Padrino application! (config/boot.rb not found)'
           end
         end
       end
@@ -51,14 +51,14 @@ module Padrino
           plugins = {}
           uri = URI.parse(PLUGIN_URL)
           http = Net::HTTP.new(uri.host, uri.port)
-          http.use_ssl = true if uri.scheme == "https"
+          http.use_ssl = true if uri.scheme == 'https'
           http.verify_mode = OpenSSL::SSL::VERIFY_NONE
           http.start do
             http.request_get(uri.path) do |res|
               plugins = res.body.scan(%r{/plugins/([-\w]+)_plugin.rb}).flatten.uniq
             end
           end
-          say "Available plugins:", :green
+          say 'Available plugins:', :green
           say plugins.map { |plugin| "  - #{plugin}" }.join("\n")
         end
       end

data/lib/padrino-gen/generators/project/config/apps.rb.tt

@@ -3,8 +3,8 @@
 # You can mount additional applications using any of these commands below:
 #
 #   Padrino.mount('blog').to('/blog')
-#   Padrino.mount('blog', :app_class => 'BlogApp').to('/blog')
-#   Padrino.mount('blog', :app_file =>  'path/to/blog/app.rb').to('/blog')
+#   Padrino.mount('blog', app_class: 'BlogApp').to('/blog')
+#   Padrino.mount('blog', app_file: 'path/to/blog/app.rb').to('/blog')
 #
 # You can also map apps to a specified host:
 #
@@ -18,7 +18,7 @@
 # By default, this file mounts the primary app which was generated with this project.
 # However, the mounted app can be modified as needed:
 #
-#   Padrino.mount('AppName', :app_file => 'path/to/file', :app_class => 'BlogApp').to('/')
+#   Padrino.mount('AppName', app_file: 'path/to/file', app_class: 'BlogApp').to('/')
 #
 
 ##
@@ -28,7 +28,7 @@
 Padrino.configure_apps do
   # enable :sessions
   set :session_secret, '<%= SecureRandom.hex(32) %>'
-  set :protection, :except => :path_traversal
+  set :protection, except: :path_traversal
   set :protect_from_csrf, true
 end
 

data/lib/padrino-gen/generators/project/config/boot.rb

@@ -1,6 +1,6 @@
 # Defines our constants
 RACK_ENV = ENV['RACK_ENV'] ||= 'development' unless defined?(RACK_ENV)
-PADRINO_ROOT = File.expand_path('../..', __FILE__) unless defined?(PADRINO_ROOT)
+PADRINO_ROOT = File.expand_path('..', __dir__) unless defined?(PADRINO_ROOT)
 
 # Load our dependencies
 require 'bundler/setup'
@@ -28,7 +28,7 @@ Bundler.require(:default, RACK_ENV)
 # ## Configure your HTML5 data helpers
 #
 # Padrino::Helpers::TagHelpers::DATA_ATTRIBUTES.push(:dialog)
-# text_field :foo, :dialog => true
+# text_field :foo, dialog: true
 # Generates: <input type="text" data-dialog="true" name="foo" />
 #
 # ## Add helpers to mailer

data/lib/padrino-gen/generators/project/config.ru

@@ -4,6 +4,6 @@
 # This file can be used to start Padrino,
 # just execute it from the command line.
 
-require File.expand_path("../config/boot.rb", __FILE__)
+require File.expand_path('config/boot.rb', __dir__)
 
 run Padrino.application

data/lib/padrino-gen/generators/project.rb

@@ -9,8 +9,8 @@ module Padrino
     class Project < Thor::Group
       Padrino::Generators.add_generator(:project, self)
 
-      def self.source_root; File.expand_path(File.dirname(__FILE__)); end
-      def self.banner; "padrino-gen project [name] [options]"; end
+      def self.source_root; __dir__; end
+      def self.banner; 'padrino-gen project [name] [options]'; end
 
       include Thor::Actions
       include Padrino::Generators::Actions
@@ -19,20 +19,20 @@ module Padrino
 
       desc "Description:\n\n\tpadrino-gen project generates a new Padrino project"
 
-      argument :name, :desc => 'The name of your padrino project'
-
-      class_option :app ,             :desc => 'The application name',                                             :aliases => '-n', :default => nil,         :type => :string
-      class_option :bundle,           :desc => 'Run bundle install',                                               :aliases => '-b', :default => false,       :type => :boolean
-      class_option :root,             :desc => 'The root destination',                                             :aliases => '-r', :default => '.',         :type => :string
-      class_option :dev,              :desc => 'Use padrino from a git checkout',                                                    :default => false,       :type => :boolean
-      class_option :tiny,             :desc => 'Generate tiny app skeleton',                                       :aliases => '-i', :default => false,       :type => :boolean
-      class_option :lean,             :desc => 'Generate lean project without apps',                               :aliases => '-l', :default => false,       :type => :boolean
-      class_option :api,              :desc => 'Generate minimal project for APIs',                                                  :default => false,       :type => :boolean
-      class_option :template,         :desc => 'Generate project from template',                                   :aliases => '-p', :default => nil,         :type => :string
-      class_option :gem,              :desc => 'Generate project as a gem',                                        :aliases => '-g', :default => false,       :type => :boolean
-      class_option :migration_format, :desc => 'Filename format for migrations (number, timestamp)',                                 :default => 'number',    :type => :string
-      class_option :adapter,          :desc => 'SQL adapter for ORM (sqlite, mysql, mysql2, mysql-gem, postgres)', :aliases => '-a', :default => 'sqlite',    :type => :string
-      class_option :server,           :desc => 'HTTP server to use (webrick, thin, puma)',                         :aliases => '-s', :default => 'webrick',   :type => :string
+      argument :name, desc: 'The name of your padrino project'
+
+      class_option :app,              desc: 'The application name',                                             aliases: '-n', default: nil,         type: :string
+      class_option :bundle,           desc: 'Run bundle install',                                               aliases: '-b', default: false,       type: :boolean
+      class_option :root,             desc: 'The root destination',                                             aliases: '-r', default: '.',         type: :string
+      class_option :dev,              desc: 'Use padrino from a git checkout',                                                 default: false,       type: :boolean
+      class_option :tiny,             desc: 'Generate tiny app skeleton',                                       aliases: '-i', default: false,       type: :boolean
+      class_option :lean,             desc: 'Generate lean project without apps',                               aliases: '-l', default: false,       type: :boolean
+      class_option :api,              desc: 'Generate minimal project for APIs',                                               default: false,       type: :boolean
+      class_option :template,         desc: 'Generate project from template',                                   aliases: '-p', default: nil,         type: :string
+      class_option :gem,              desc: 'Generate project as a gem',                                        aliases: '-g', default: false,       type: :boolean
+      class_option :migration_format, desc: 'Filename format for migrations (number, timestamp)',                              default: 'number',    type: :string
+      class_option :adapter,          desc: 'SQL adapter for ORM (sqlite, mysql, mysql2, mysql-gem, postgres)', aliases: '-a', default: 'sqlite',    type: :string
+      class_option :server,           desc: 'HTTP server to use (webrick, thin, puma)',                         aliases: '-s', default: 'webrick',   type: :string
 
       # Definitions for the available customizable components.
       defines_component_options
@@ -45,11 +45,11 @@ module Padrino
       #
       def setup_project
         valid_constant? name
-        app = (options[:app] || "App")
+        app = options[:app] || 'App'
 
         @project_name = name.gsub(/\W/, '_').underscore.camelize
 
-        fail "Constant `#{@project_name}` already exists. Please, use another name" if already_exists?(@project_name)
+        raise "Constant `#{@project_name}` already exists. Please, use another name" if already_exists?(@project_name)
 
         @app_name = app.gsub(/\W/, '_').camelize
         self.destination_root = File.join(options[:root], name)
@@ -63,15 +63,15 @@ module Padrino
           store_component_config('.components')
           unless options[:lean]
             app_skeleton('app', options[:tiny])
-            append_file destination_root('config/apps.rb'), "Padrino.mount('#{@project_name}::#{@app_name}', :app_file => Padrino.root('app/app.rb')).to('/')\n"
+            append_file destination_root('config/apps.rb'), "Padrino.mount('#{@project_name}::#{@app_name}', app_file: Padrino.root('app/app.rb')).to('/')\n"
           end
           template 'templates/Gemfile.tt', destination_root('Gemfile')
           template 'templates/Rakefile.tt', destination_root('Rakefile')
           template 'templates/project_bin.tt', destination_root("exe/#{name}")
-          File.chmod(0755, destination_root("exe/#{name}"))
+          File.chmod(0o755, destination_root("exe/#{name}"))
           if options.gem?
-            template 'templates/gem/gemspec.tt', destination_root(name + '.gemspec')
-            inject_into_file destination_root('Rakefile'), "require 'bundler/gem_tasks'\n", :after => "require 'bundler/setup'\n"
+            template 'templates/gem/gemspec.tt', destination_root("#{name}.gemspec")
+            inject_into_file destination_root('Rakefile'), "require 'bundler/gem_tasks'\n", after: "require 'bundler/setup'\n"
             template 'templates/gem/README.md.tt', destination_root('README.md')
             template 'templates/gem/lib/libname.tt', destination_root("lib/#{name}.rb")
             template 'templates/gem/lib/libname/version.tt', destination_root("lib/#{name}/version.rb")
@@ -87,12 +87,12 @@ module Padrino
       #
       def setup_components
         return if options[:template]
-        @_components = options.class.new options.select{ |key,_| self.class.component_types.include?(key.to_sym) }
+        @_components = options.class.new(options.slice(*self.class.component_types))
         self.class.component_types.each do |comp|
           choice = @_components[comp] = resolve_valid_choice(comp)
           execute_component_setup(comp, choice)
         end
-        store_component_config('.components', :force => true)
+        store_component_config('.components', force: true)
         store_component_choice(:namespace, @project_name)
         store_component_choice(:migration_format, options[:migration_format])
       end
@@ -101,35 +101,33 @@ module Padrino
       # Generates test files for tiny app skeleton.
       #
       def setup_test_files
-        if options[:tiny] && @_components[:test] != :none
-          test_component = @_components[:test]
-          test_component = "rspec" if test_component == "cucumber"
-          uppercase_test_component = test_component.upcase
-          controller_template_name = "#{uppercase_test_component}_CONTROLLER_TEST"
-          helper_template_name     = "#{uppercase_test_component}_HELPER_TEST"
-          return unless defined?(controller_template_name)
-
-          controller_content = instance_eval(controller_template_name).gsub(/!PATH!/, "Controller").gsub(/!NAME!/, "").gsub(/!EXPANDED_PATH!/, "/")
-          helper_content     = instance_eval(helper_template_name).gsub(/!NAME!/, "#{@project_name}::#{@app_name}::#{DEFAULT_HELPER_NAME}")
-
-          proc{|*args| args.map{|str| str.gsub!(/!PATH!/, recognize_path)} }.call(controller_content, helper_content)
-
-          directory_name = [:rspec].include?(test_component.to_sym) ? "spec" : "test"
-          base_path      = File.join(directory_name, "app")
-          create_file destination_root("#{base_path}/controllers/controllers_#{directory_name}.rb"), controller_content, :skip => true
-          create_file destination_root("#{base_path}/helpers/helpers_#{directory_name}.rb"),         helper_content,     :skip => true
-          helper_path = destination_root(File.join(directory_name, "#{directory_name == "spec" ? "spec_helper" : "test_config"}.rb"))
-          gsub_file helper_path, %r{helpers/\*\*/\*\.rb}, "helpers.rb"
-        end
+        return unless options[:tiny] && @_components[:test] != :none
+
+        test_component = @_components[:test]
+        test_component = 'rspec' if test_component == 'cucumber'
+        uppercase_test_component = test_component.upcase
+        controller_template_name = "#{uppercase_test_component}_CONTROLLER_TEST"
+        helper_template_name     = "#{uppercase_test_component}_HELPER_TEST"
+        return unless defined?(controller_template_name)
+
+        controller_content = instance_eval(controller_template_name).gsub(/!PATH!/, 'Controller').gsub(/!NAME!/, '').gsub(/!EXPANDED_PATH!/, '/')
+        helper_content     = instance_eval(helper_template_name).gsub(/!NAME!/, "#{@project_name}::#{@app_name}::#{DEFAULT_HELPER_NAME}")
+
+        proc { |*args| args.map { |str| str.gsub!(/!PATH!/, recognize_path) } }.call(controller_content, helper_content)
+
+        directory_name = [:rspec].include?(test_component.to_sym) ? 'spec' : 'test'
+        base_path      = File.join(directory_name, 'app')
+        create_file destination_root("#{base_path}/controllers/controllers_#{directory_name}.rb"), controller_content, skip: true
+        create_file destination_root("#{base_path}/helpers/helpers_#{directory_name}.rb"),         helper_content,     skip: true
+        helper_path = destination_root(File.join(directory_name, "#{directory_name == 'spec' ? 'spec_helper' : 'test_config'}.rb"))
+        gsub_file helper_path, %r{helpers/\*\*/\*\.rb}, 'helpers.rb'
       end
 
       ##
       # Bundle all required components using bundler and Gemfile.
       #
       def bundle_dependencies
-        if options[:bundle]
-          run_bundler
-        end
+        run_bundler if options[:bundle]
       end
 
       ##
@@ -141,8 +139,8 @@ module Padrino
         say "#{name} is ready for development!", :green
         say '=' * 65, :green
         say "$ cd #{options[:root]}/#{name}"
-        say "$ bundle --binstubs" unless options[:bundle]
-        say "=" * 65, :green
+        say '$ bundle --binstubs' unless options[:bundle]
+        say '=' * 65, :green
         say
       end
 
@@ -151,7 +149,7 @@ module Padrino
       #
       def git_author_name
         git_author_name = `git config user.name`.chomp rescue ''
-        git_author_name.empty? ? "TODO: Write your name" : git_author_name
+        git_author_name.empty? ? 'TODO: Write your name' : git_author_name
       end
 
       ##
@@ -159,7 +157,7 @@ module Padrino
       #
       def git_author_email
         git_author_email = `git config user.email`.chomp rescue ''
-        git_author_email.empty? ? "TODO: Write your email address" : git_author_email
+        git_author_email.empty? ? 'TODO: Write your email address' : git_author_email
       end
     end
   end

data/lib/padrino-gen/generators/runner.rb

@@ -8,20 +8,19 @@ module Padrino
     # common actions for modifying a project or application.
     #
     module Runner
-
       # Generates project scaffold based on a given template file.
       #
       # @param [Hash] options
       #   Options to use to generate the project.
       #
       # @example
-      #   project :test => :shoulda, :orm => :activerecord, :renderer => "haml"
+      #   project test: :shoulda, orm: :activerecord, renderer: "haml"
       #
-      def project(options={})
-        components = options.sort_by { |k, v| k.to_s }.map { |component, value| "--#{component}=#{value}" }
-        params = [name, *components].push("-r=#{destination_root("../")}")
-        say "=> Executing: padrino-gen project #{params.join(" ")}", :magenta
-        Padrino.bin_gen(*params.unshift("project"))
+      def project(options = {})
+        components = options.sort_by { |k, _v| k.to_s }.map { |component, value| "--#{component}=#{value}" }
+        params = [name, *components].push("-r=#{destination_root('../')}")
+        say "=> Executing: padrino-gen project #{params.join(' ')}", :magenta
+        Padrino.bin_gen(*params.unshift('project'))
       end
 
       ##
@@ -37,10 +36,10 @@ module Padrino
       #   generate :controller, "posts get:index get:new post:new"
       #   generate :migration, "AddEmailToUser email:string"
       #
-      def generate(type, arguments="")
-        params = arguments.split(" ").push("-r=#{destination_root}")
+      def generate(type, arguments = '')
+        params = arguments.split(' ').push("-r=#{destination_root}")
         params.push("--app=#{@_app_name}") if @_app_name
-        say "=> Executing: padrino-gen #{type} #{params.join(" ")}", :magenta
+        say "=> Executing: padrino-gen #{type} #{params.join(' ')}", :magenta
         Padrino.bin_gen(*params.unshift(type))
       end
 
@@ -54,7 +53,7 @@ module Padrino
       #   rake "custom task1 task2"
       #
       def rake(command)
-        Padrino.bin("rake", command, "-c=#{destination_root}")
+        Padrino.bin('rake', command, "-c=#{destination_root}")
       end
 
       ##
@@ -74,11 +73,12 @@ module Padrino
       def app(name)
         say "=> Executing: padrino-gen app #{name} -r=#{destination_root}", :magenta
         Padrino.bin_gen(:app, name.to_s, "-r=#{destination_root}")
-        if block_given?
-          @_app_name = name
-          yield
-          @_app_name = nil
-        end
+        return unless block_given?
+
+        @_app_name = name
+        yield
+      ensure
+        @_app_name = nil
       end
 
       ##
@@ -96,7 +96,7 @@ module Padrino
       #
       def git(*args)
         FileUtils.cd(destination_root) do
-          cmd = "git %s" % args.join(' ')
+          cmd = format('git %s', args.join(' '))
           say cmd, :green
           system cmd
         end
@@ -122,21 +122,22 @@ module Padrino
       def execute_runner(kind, template_file)
         # Determine resolved template path
         template_file = template_file.to_s
-        template_path = case
-          when template_file =~ %r{^https?://} && template_file !~ /gist/
+        template_path =
+          if template_file =~ %r{^https?://} && template_file !~ /gist/
             template_file
-          when template_file =~ /gist/ && template_file !~ /raw/
-            raw_link, _ = *URI.open(template_file) { |io| io.read.scan(/<a\s+href\s?\=\"(.*?)\"\>raw/) }
+          elsif template_file =~ /gist/ && template_file !~ /raw/
+            raw_link, = *URI.open(template_file) { |io| io.read.scan(/<a\s+href\s?="(.*?)">raw/) }
             raw_link ? "https://gist.github.com#{raw_link[0]}" : template_file
-          when File.extname(template_file).empty? # referencing official plugin (i.e hoptoad)
+          elsif File.extname(template_file).empty? # referencing official plugin (i.e hoptoad)
             "https://raw.github.com/padrino/padrino-recipes/master/#{kind.to_s.pluralize}/#{template_file}_#{kind}.rb"
           else # local file on system
             File.expand_path(template_file)
           end
+
         begin
           self.apply(template_path)
-        rescue => error
-          say("The template at #{template_path} could not be loaded: #{error.message}", :red)
+        rescue StandardError => e
+          say("The template at #{template_path} could not be loaded: #{e.message}", :red)
         end
       end
     end