sinatra 2.2.4 → 3.0.0

data/Rakefile

@@ -1,15 +1,17 @@
+# frozen_string_literal: true
+
 require 'rake/clean'
 require 'rake/testtask'
 require 'fileutils'
 require 'date'
 
-task :default => :test
-task :spec => :test
+task default: :test
+task spec: :test
 
-CLEAN.include "**/*.rbc"
+CLEAN.include '**/*.rbc'
 
 def source_version
-  @source_version ||= File.read(File.expand_path("../VERSION", __FILE__)).strip
+  @source_version ||= File.read(File.expand_path('VERSION', __dir__)).strip
 end
 
 def prev_feature
@@ -17,32 +19,28 @@ def prev_feature
 end
 
 def prev_version
-  return prev_feature + '.0' if source_version.end_with? '.0'
+  return "#{prev_feature}.0" if source_version.end_with? '.0'
+
   source_version.gsub(/\d+$/) { |s| s.to_i - 1 }
 end
 
 # SPECS ===============================================================
 
-task :test do
-  ENV['LANG'] = 'C'
-  ENV.delete 'LC_CTYPE'
-end
-
 Rake::TestTask.new(:test) do |t|
   t.test_files = FileList['test/*_test.rb']
   t.ruby_opts = ['-r rubygems'] if defined? Gem
-  t.ruby_opts << '-I.'
   t.warning = true
 end
 
-Rake::TestTask.new(:"test:core") do |t|
-  core_tests = %w[base delegator encoding extensions filter
-     helpers mapped_error middleware radius rdoc
-     readme request response result route_added_hook
-     routing server settings sinatra static templates]
-  t.test_files = core_tests.map {|n| "test/#{n}_test.rb"}
-  t.ruby_opts = ["-r rubygems"] if defined? Gem
-  t.ruby_opts << "-I."
+Rake::TestTask.new(:'test:core') do |t|
+  core_tests = %w[
+    base delegator encoding extensions filter
+    helpers mapped_error middleware rdoc
+    readme request response result route_added_hook
+    routing server settings sinatra static templates
+  ]
+  t.test_files = core_tests.map { |n| "test/#{n}_test.rb" }
+  t.ruby_opts = ['-r rubygems'] if defined? Gem
   t.warning = true
 end
 
@@ -51,7 +49,7 @@ end
 namespace :test do
   desc 'Measures test coverage'
   task :coverage do
-    rm_f "coverage"
+    rm_f 'coverage'
     ENV['COVERAGE'] = '1'
     Rake::Task['test'].invoke
   end
@@ -60,26 +58,26 @@ end
 # Website =============================================================
 
 desc 'Generate RDoc under doc/api'
-task 'doc'     => ['doc:api']
-task('doc:api') { sh "yardoc -o doc/api" }
+task 'doc' => ['doc:api']
+task('doc:api') { sh 'yardoc -o doc/api' }
 CLEAN.include 'doc/api'
 
 # README ===============================================================
 
-task :add_template, [:name] do |t, args|
+task :add_template, [:name] do |_t, args|
   Dir.glob('README.*') do |file|
     code = File.read(file)
     if code =~ /^===.*#{args.name.capitalize}/
       puts "Already covered in #{file}"
     else
-      template = code[/===[^\n]*Liquid.*index\.liquid<\/tt>[^\n]*/m]
-      if !template
-        puts "Liquid not found in #{file}"
-      else
+      template = code[%r{===[^\n]*Liquid.*index\.liquid</tt>[^\n]*}m]
+      if template
         puts "Adding section to #{file}"
         template = template.gsub(/Liquid/, args.name.capitalize).gsub(/liquid/, args.name.downcase)
         code.gsub! /^(\s*===.*CoffeeScript)/, "\n" << template << "\n\\1"
-        File.open(file, "w") { |f| f << code }
+        File.open(file, 'w') { |f| f << code }
+      else
+        puts "Liquid not found in #{file}"
       end
     end
   end
@@ -87,29 +85,31 @@ end
 
 # Thanks in announcement ===============================================
 
-team = ["Ryan Tomayko", "Blake Mizerany", "Simon Rozet", "Konstantin Haase", "Zachary Scott"]
-desc "list of contributors"
-task :thanks, ['release:all', :backports] do |t, a|
-  a.with_defaults :release => "#{prev_version}..HEAD",
-    :backports => "#{prev_feature}.0..#{prev_feature}.x"
+team = ['Ryan Tomayko', 'Blake Mizerany', 'Simon Rozet', 'Konstantin Haase', 'Zachary Scott']
+desc 'list of contributors'
+task :thanks, ['release:all', :backports] do |_t, a|
+  a.with_defaults release: "#{prev_version}..HEAD",
+                  backports: "#{prev_feature}.0..#{prev_feature}.x"
+
   included = `git log --format=format:"%aN\t%s" #{a.release}`.lines.map { |l| l.force_encoding('binary') }
   excluded = `git log --format=format:"%aN\t%s" #{a.backports}`.lines.map { |l| l.force_encoding('binary') }
   commits  = (included - excluded).group_by { |c| c[/^[^\t]+/] }
   authors  = commits.keys.sort_by { |n| - commits[n].size } - team
-  puts authors[0..-2].join(', ') << " and " << authors.last,
-    "(based on commits included in #{a.release}, but not in #{a.backports})"
+  puts authors[0..-2].join(', ') << ' and ' << authors.last,
+       "(based on commits included in #{a.release}, but not in #{a.backports})"
 end
 
-desc "list of authors"
-task :authors, [:commit_range, :format, :sep] do |t, a|
-  a.with_defaults :format => "%s (%d)", :sep => ", ", :commit_range => '--all'
+desc 'list of authors'
+task :authors, [:commit_range, :format, :sep] do |_t, a|
+  a.with_defaults format: '%s (%d)', sep: ', ', commit_range: '--all'
   authors = Hash.new(0)
-  blake   = "Blake Mizerany"
+  blake   = 'Blake Mizerany'
   overall = 0
   mapping = {
-    "blake.mizerany@gmail.com" => blake, "bmizerany" => blake,
-    "a_user@mac.com" => blake, "ichverstehe" => "Harry Vangberg",
-    "Wu Jiang (nouse)" => "Wu Jiang" }
+    'blake.mizerany@gmail.com' => blake, 'bmizerany' => blake,
+    'a_user@mac.com' => blake, 'ichverstehe' => 'Harry Vangberg',
+    'Wu Jiang (nouse)' => 'Wu Jiang'
+  }
   `git shortlog -s #{a.commit_range}`.lines.map do |line|
     line = line.force_encoding 'binary' if line.respond_to? :force_encoding
     num, name = line.split("\t", 2).map(&:strip)
@@ -117,18 +117,18 @@ task :authors, [:commit_range, :format, :sep] do |t, a|
     overall += num.to_i
   end
   puts "#{overall} commits by #{authors.count} authors:"
-  puts authors.sort_by { |n,c| -c }.map { |e| a.format % e }.join(a.sep)
+  puts authors.sort_by { |_n, c| -c }.map { |e| a.format % e }.join(a.sep)
 end
 
-desc "generates TOC"
-task :toc, [:readme] do |t, a|
-  a.with_defaults :readme => 'README.md'
+desc 'generates TOC'
+task :toc, [:readme] do |_t, a|
+  a.with_defaults readme: 'README.md'
 
   def self.link(title)
     title.downcase.gsub(/(?!-)\W /, '-').gsub(' ', '-').gsub(/(?!-)\W/, '')
   end
 
-  puts "* [Sinatra](#sinatra)"
+  puts '* [Sinatra](#sinatra)'
   title = Regexp.new('(?<=\* )(.*)') # so Ruby 1.8 doesn't complain
   File.binread(a.readme).scan(/^##.*/) do |line|
     puts line.gsub(/#(?=#)/, '    ').gsub('#', '*').gsub(title) { "[#{$1}](##{link($1)})" }
@@ -139,12 +139,12 @@ end
 
 if defined?(Gem)
   GEMS_AND_ROOT_DIRECTORIES = {
-    "sinatra" => ".",
-    "sinatra-contrib" => "./sinatra-contrib",
-    "rack-protection" => "./rack-protection"
-  }
+    'sinatra' => '.',
+    'sinatra-contrib' => './sinatra-contrib',
+    'rack-protection' => './rack-protection'
+  }.freeze
 
-  def package(gem, ext='')
+  def package(gem, ext = '')
     "pkg/#{gem}-#{source_version}" + ext
   end
 
@@ -152,12 +152,12 @@ if defined?(Gem)
   CLOBBER.include('pkg')
 
   GEMS_AND_ROOT_DIRECTORIES.each do |gem, directory|
-    file package(gem, '.gem') => ["pkg/", "#{directory + '/' + gem}.gemspec"] do |f|
+    file package(gem, '.gem') => ['pkg/', "#{"#{directory}/#{gem}"}.gemspec"] do |f|
       sh "cd #{directory} && gem build #{gem}.gemspec"
-      mv directory + "/" + File.basename(f.name), f.name
+      mv "#{directory}/#{File.basename(f.name)}", f.name
     end
 
-    file package(gem, '.tar.gz') => ["pkg/"] do |f|
+    file package(gem, '.tar.gz') => ['pkg/'] do |f|
       sh <<-SH
         git archive \
           --prefix=#{gem}-#{source_version}/ \
@@ -168,29 +168,29 @@ if defined?(Gem)
   end
 
   namespace :package do
-    GEMS_AND_ROOT_DIRECTORIES.each do |gem, directory|
+    GEMS_AND_ROOT_DIRECTORIES.each do |gem, _directory|
       desc "Build #{gem} packages"
       task gem => %w[.gem .tar.gz].map { |e| package(gem, e) }
     end
 
-    desc "Build all packages"
-    task :all => GEMS_AND_ROOT_DIRECTORIES.keys
+    desc 'Build all packages'
+    task all: GEMS_AND_ROOT_DIRECTORIES.keys
   end
 
   namespace :install do
-    GEMS_AND_ROOT_DIRECTORIES.each do |gem, directory|
+    GEMS_AND_ROOT_DIRECTORIES.each do |gem, _directory|
       desc "Build and install #{gem} as local gem"
       task gem => package(gem, '.gem') do
         sh "gem install #{package(gem, '.gem')}"
       end
     end
 
-    desc "Build and install all of the gems as local gems"
-    task :all => GEMS_AND_ROOT_DIRECTORIES.keys
+    desc 'Build and install all of the gems as local gems'
+    task all: GEMS_AND_ROOT_DIRECTORIES.keys
   end
 
   namespace :release do
-    GEMS_AND_ROOT_DIRECTORIES.each do |gem, directory|
+    GEMS_AND_ROOT_DIRECTORIES.each do |gem, _directory|
       desc "Release #{gem} as a package"
       task gem => "package:#{gem}" do
         sh <<-SH
@@ -200,7 +200,7 @@ if defined?(Gem)
       end
     end
 
-    desc "Commits the version to github repository"
+    desc 'Commits the version to github repository'
     task :commit_version do
       %w[
         lib/sinatra
@@ -208,18 +208,18 @@ if defined?(Gem)
         rack-protection/lib/rack/protection
       ].each do |path|
         path = File.join(path, 'version.rb')
-        File.write(path, File.read(path).sub(/VERSION = '(.+?)'/, "VERSION = '#{source_version}'"))
+        #File.write(path, File.read(path).sub(/VERSION = '(.+?)'/, "VERSION = '#{source_version}'"))
       end
 
+      #git commit --allow-empty -a -m '#{source_version} release'  &&
       sh <<-SH
-        git commit --allow-empty -a -m '#{source_version} release'  &&
         git tag -s v#{source_version} -m '#{source_version} release'  &&
         git push && (git push origin || true) &&
         git push --tags && (git push origin --tags || true)
       SH
     end
 
-    desc "Release all gems as packages"
-    task :all => [:test, :commit_version] + GEMS_AND_ROOT_DIRECTORIES.keys
+    desc 'Release all gems as packages'
+    task all: %i[test commit_version] + GEMS_AND_ROOT_DIRECTORIES.keys
   end
 end

data/VERSION

@@ -1 +1 @@
-2.2.4
+3.0.0

data/examples/chat.rb

@@ -1,16 +1,18 @@
 #!/usr/bin/env ruby -I ../lib -I lib
-# coding: utf-8
+# frozen_string_literal: true
+
 require_relative 'rainbows'
+
 require 'sinatra'
 set :server, :rainbows
 connections = []
 
 get '/' do
   halt erb(:login) unless params[:user]
-  erb :chat, :locals => { :user => params[:user].gsub(/\W/, '') }
+  erb :chat, locals: { user: params[:user].gsub(/\W/, '') }
 end
 
-get '/stream', :provides => 'text/event-stream' do
+get '/stream', provides: 'text/event-stream' do
   stream :keep_open do |out|
     connections << out
     out.callback { connections.delete(out) }

data/examples/rainbows.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rainbows'
 
 module Rack
@@ -8,7 +10,7 @@ module Rack
           listeners: ["#{options[:Host]}:#{options[:Port]}"],
           worker_processes: 1,
           timeout: 30,
-          config_file: ::File.expand_path('rainbows.conf', __dir__),
+          config_file: ::File.expand_path('rainbows.conf', __dir__)
         }
 
         ::Rainbows::HttpServer.new(app, rainbows_options).start.join

data/examples/simple.rb

@@ -1,3 +1,5 @@
 #!/usr/bin/env ruby -I ../lib -I lib
+# frozen_string_literal: true
+
 require 'sinatra'
 get('/') { 'this is a simple app' }

data/examples/stream.ru

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # this example does *not* work properly with WEBrick
 #
 # run *one* of these: