Drop last vestiges of ruby 2.5 support

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index e818884ae85d630ae5f38aa1ea9e0a23113f4852..d6a06b549cf73125ae1eccebbc58c1bde732b6f4 100644 (file)
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -8,7 +8,7 @@ jobs:
     strategy:
       matrix:
         ubuntu: [18.04, 20.04]
-        ruby: [2.5, 2.7, 3.0]
+        ruby: [2.7, 3.0]
     runs-on: ubuntu-${{ matrix.ubuntu }}
     env:
       RAILS_ENV: test

diff --git a/.rubocop.yml b/.rubocop.yml
index 1931eb55ff73af4121df66e05b96f90b383f0c4f..49fba1d0c64ac68501da202c0bad105599bdbe1f 100644 (file)
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -7,7 +7,7 @@ require:
   - rubocop-rake
 
 AllCops:
-  TargetRubyVersion: 2.5
+  TargetRubyVersion: 2.7
   NewCops: enable
   Exclude:
     - 'vendor/**/*'

diff --git a/app/helpers/user_roles_helper.rb b/app/helpers/user_roles_helper.rb
index 79e7cc012ad1e2c32e25a1f9df21c20bb983c4dc..15231f4d066206575e457e3a7d3a2a7b5ccc2796 100644 (file)
--- a/app/helpers/user_roles_helper.rb
+++ b/app/helpers/user_roles_helper.rb
@@ -1,6 +1,6 @@
 module UserRolesHelper
   def role_icons(user)
-    safe_join(UserRole::ALL_ROLES.collect { |role| role_icon(user, role) }.compact, " ")
+    safe_join(UserRole::ALL_ROLES.filter_map { |role| role_icon(user, role) }, " ")
   end
 
   def role_icon(user, role)

diff --git a/config/initializers/oauth.rb b/config/initializers/oauth.rb
index 9179e431f9558d7483e9980311a150c0451e8855..57b89564ffc239bef207abf47fe0abadfe9d51b0 100644 (file)
--- a/config/initializers/oauth.rb
+++ b/config/initializers/oauth.rb
@@ -16,7 +16,7 @@ module OAuth
 
     def unescape(value)
       value.to_s.gsub(/%\h{2}/) do |c|
-        c[1..-1].to_i(16).chr
+        c[1..].to_i(16).chr
       end.force_encoding(Encoding::UTF_8)
     end
   end
@@ -52,12 +52,12 @@ module OpenStreetMap
     module ClassMethods
       def included(controller)
         controller.class_eval do
-          def self.before_filter(*names, &blk)
-            before_action(*names, &blk)
+          def self.before_filter(...)
+            before_action(...)
           end
 
-          def self.skip_before_filter(*names, &blk)
-            skip_before_action(*names, &blk)
+          def self.skip_before_filter(...)
+            skip_before_action(...)
           end
         end
 

diff --git a/lib/id.rb b/lib/id.rb
index 4e6cf3b7c8475e403da4322677f9dbb40c27f189..87c4a194e9ff5d4b1889aa4d583e1383a00af122 100644 (file)
--- a/lib/id.rb
+++ b/lib/id.rb
@@ -1,3 +1,3 @@
 module ID
-  LOCALES = Locale.list(Rails.root.join("vendor/assets/iD/iD/locales").entries.map { |p| p.basename.to_s[/(.*).json/] && Regexp.last_match(1) }.compact)
+  LOCALES = Locale.list(Rails.root.join("vendor/assets/iD/iD/locales").entries.filter_map { |p| p.basename.to_s[/(.*).json/] && Regexp.last_match(1) })
 end