From b71eb4f6ec8ad2250a5065ffe76078c44e8b2213 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Mon, 20 Jan 2025 15:43:32 +0200 Subject: [PATCH 01/12] Add support for Rails 6.1.0 patches --- carpentry.gemspec | 4 ++-- lib/carpentry/version.rb | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/carpentry.gemspec b/carpentry.gemspec index 4a77cd2..1c08ff6 100644 --- a/carpentry.gemspec +++ b/carpentry.gemspec @@ -17,10 +17,10 @@ Gem::Specification.new do |s| s.files = Dir["{app,lib,config}/**/*"] + ["Rakefile", "Gemfile", "README.md"] - s.add_dependency("railties", ">= 4.2.0", "<= 6.1.4.1") + s.add_dependency("railties", ">= 4.2.0", "~> 6.1.0") s.add_development_dependency "rdoc" - s.add_development_dependency "rails", ">= 4.2.0", "<= 6.1.4.1" + s.add_development_dependency "rails", ">= 4.2.0", "~> 6.1.0" s.add_development_dependency "capybara", "~> 2.13" s.add_development_dependency "autotest-standalone" s.add_development_dependency "rspec-rails", "~> 3.4" diff --git a/lib/carpentry/version.rb b/lib/carpentry/version.rb index 97d19cf..a3692ab 100644 --- a/lib/carpentry/version.rb +++ b/lib/carpentry/version.rb @@ -1,3 +1,3 @@ module Carpentry - VERSION = "2.2.5" + VERSION = "2.2.6" end From 8754b8bd07578b70253cf6a10075ea83dc981798 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Mon, 20 Jan 2025 15:44:06 +0200 Subject: [PATCH 02/12] Add Ruby 3.4. for testing --- .travis.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 0225817..f4d5731 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,6 +6,7 @@ rvm: - 2.6 - 2.7 - 3.0 + - 3.4 gemfile: - Gemfile - Gemfile.rails-5.0.x @@ -16,11 +17,15 @@ jobs: exclude: - rvm: 3.0 gemfile: Gemfile.rails-5.0.x + - rvm: 3.4 + gemfile: Gemfile.rails-5.0.x - rvm: 3.0 gemfile: Gemfile.rails-5.1.x + - rvm: 3.4 + gemfile: Gemfile.rails-5.1.x script: "./bin/rspec spec" before_install: - - gem install bundler + - gem install bundler -v 2.4.22 notifications: email: false deploy: From d738e60389585bf7467cbae2e9211a6be3dd9f9b Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Tue, 21 Jan 2025 08:54:23 +0200 Subject: [PATCH 03/12] Fix dependency concurrent-ruby to v1.3.5 Concurrent-ruby v1.3.5 has removed the dependency on logger. All Rails versions before 7.1 are affected by this issue because they have a direct dependency on concurrent --- carpentry.gemspec | 1 + 1 file changed, 1 insertion(+) diff --git a/carpentry.gemspec b/carpentry.gemspec index 1c08ff6..5e7886d 100644 --- a/carpentry.gemspec +++ b/carpentry.gemspec @@ -21,6 +21,7 @@ Gem::Specification.new do |s| s.add_development_dependency "rdoc" s.add_development_dependency "rails", ">= 4.2.0", "~> 6.1.0" + s.add_development_dependency "concurrent-ruby", "<= 1.3.4" s.add_development_dependency "capybara", "~> 2.13" s.add_development_dependency "autotest-standalone" s.add_development_dependency "rspec-rails", "~> 3.4" From 4b4036bfec8bd91a05fc7ddc47e73fac3dcfedff Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Tue, 21 Jan 2025 13:23:10 +0200 Subject: [PATCH 04/12] Change travis bundler version --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index f4d5731..96c8bdf 100644 --- a/.travis.yml +++ b/.travis.yml @@ -25,7 +25,7 @@ jobs: gemfile: Gemfile.rails-5.1.x script: "./bin/rspec spec" before_install: - - gem install bundler -v 2.4.22 + - gem install bundler -v 2.0 notifications: email: false deploy: From 4512a139962275040bea53075ad9cde366995b97 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 10:00:56 +0200 Subject: [PATCH 05/12] Change exists method to singular form Support for Ruby 3.2.0 - Alias exists? (pluralized) for exist? (singular) has been removed. --- spec/dummy/config/boot.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spec/dummy/config/boot.rb b/spec/dummy/config/boot.rb index ef36047..6266cfc 100644 --- a/spec/dummy/config/boot.rb +++ b/spec/dummy/config/boot.rb @@ -1,5 +1,5 @@ # Set up gems listed in the Gemfile. ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../../../Gemfile', __FILE__) -require 'bundler/setup' if File.exists?(ENV['BUNDLE_GEMFILE']) +require 'bundler/setup' if File.exist?(ENV['BUNDLE_GEMFILE']) $LOAD_PATH.unshift File.expand_path('../../../../lib', __FILE__) From a3419004c66cddf5055a22bbfd13780c8aa71741 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 11:30:30 +0200 Subject: [PATCH 06/12] Ruby 3.4 dropping support for BigDecimal and mutex_m --- carpentry.gemspec | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/carpentry.gemspec b/carpentry.gemspec index 5e7886d..a0230fc 100644 --- a/carpentry.gemspec +++ b/carpentry.gemspec @@ -24,5 +24,10 @@ Gem::Specification.new do |s| s.add_development_dependency "concurrent-ruby", "<= 1.3.4" s.add_development_dependency "capybara", "~> 2.13" s.add_development_dependency "autotest-standalone" - s.add_development_dependency "rspec-rails", "~> 3.4" + s.add_development_dependency "rspec-rails" + + if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('3.4') + s.add_development_dependency "bigdecimal" + s.add_development_dependency "mutex_m" + end end From 2fa8d802bbfec7d54dad6e03b830e5e16ccaa1be Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 11:43:22 +0200 Subject: [PATCH 07/12] Change capybara dev dependency version requirement --- carpentry.gemspec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/carpentry.gemspec b/carpentry.gemspec index a0230fc..b2bd5b3 100644 --- a/carpentry.gemspec +++ b/carpentry.gemspec @@ -22,7 +22,7 @@ Gem::Specification.new do |s| s.add_development_dependency "rdoc" s.add_development_dependency "rails", ">= 4.2.0", "~> 6.1.0" s.add_development_dependency "concurrent-ruby", "<= 1.3.4" - s.add_development_dependency "capybara", "~> 2.13" + s.add_development_dependency "capybara", ">= 2.13" s.add_development_dependency "autotest-standalone" s.add_development_dependency "rspec-rails" From a2605b5ddb2adb26c3eaa16e2a416f2331a72a20 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 12:33:24 +0200 Subject: [PATCH 08/12] Add drb as dev dependency for ruby 3.4. --- carpentry.gemspec | 1 + 1 file changed, 1 insertion(+) diff --git a/carpentry.gemspec b/carpentry.gemspec index b2bd5b3..1ada44f 100644 --- a/carpentry.gemspec +++ b/carpentry.gemspec @@ -29,5 +29,6 @@ Gem::Specification.new do |s| if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('3.4') s.add_development_dependency "bigdecimal" s.add_development_dependency "mutex_m" + s.add_development_dependency "drb" end end From c2c0eb304199cded547b73585a474123985202b4 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 10:24:34 +0200 Subject: [PATCH 09/12] Add github actions --- .github/workflows/ci.yml | 53 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 53 insertions(+) create mode 100644 .github/workflows/ci.yml diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml new file mode 100644 index 0000000..c5ceaeb --- /dev/null +++ b/.github/workflows/ci.yml @@ -0,0 +1,53 @@ +name: CI + +concurrency: + group: ${{ github.workflow }}-${{ github.ref_name }} + cancel-in-progress: true + +on: + push: + branches: + - kiskolabs/add_rails_6.1_and_ruby_3_4 + + pull_request: + branches: + - kiskolabs/add_rails_6.1_and_ruby_3_4 + +jobs: + test: + runs-on: ubuntu-latest + + strategy: + matrix: + ruby: ['2.6', '2.7', '3.0', '3.4'] + gemfile: + - Gemfile + - Gemfile.rails-5.0.x + - Gemfile.rails-5.1.x + - Gemfile.rails-6.0.x + - Gemfile.rails-6.1.x + + exclude: + - ruby: 3.0 + gemfile: Gemfile.rails-5.0.x + - ruby: 3.4 + gemfile: Gemfile.rails-5.0.x + - ruby: 3.0 + gemfile: Gemfile.rails-5.1.x + - ruby: 3.4 + gemfile: Gemfile.rails-5.1.x + + steps: + - name: Checkout code + uses: actions/checkout@v4 + - name: Set up Ruby + uses: ruby/setup-ruby@v1 + with: + ruby-version: ${{ matrix.ruby }} + bundler: '2.4' + bundler-cache: true + cache-version: 1 # ignore contents of the cache and get and build all the gems anew. + + - name: Run tests + run: | + bundle exec rspec From a43136e4c97875af9e4cc609d9c05f47778dd606 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 13:39:24 +0200 Subject: [PATCH 10/12] Create initial publish github actions --- .github/workflows/ci.yml | 1 + .github/workflows/publish.yml | 33 +++++++++++++++++++++++++++++++++ 2 files changed, 34 insertions(+) create mode 100644 .github/workflows/publish.yml diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c5ceaeb..3b4423d 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -5,6 +5,7 @@ concurrency: cancel-in-progress: true on: + workflow_call: push: branches: - kiskolabs/add_rails_6.1_and_ruby_3_4 diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml new file mode 100644 index 0000000..3f48cee --- /dev/null +++ b/.github/workflows/publish.yml @@ -0,0 +1,33 @@ +name: Deploy to RubyGems + +on: + workflow_dispatch: + push: + tags: + - '*' + branches: + - kiskolabs/gha-publish-to-rubygems + +jobs: + test: + uses: ./.github/workflows/ci.yml + push: + runs-on: ubuntu-latest + needs: [test] + permissions: + contents: write + id-token: write + steps: + - name: Test + run: | + echo "Testing successful case" + + # - uses: actions/checkout@v4 + # - name: Set up Ruby + # uses: ruby/setup-ruby@v1 + # with: + # bundler-cache: true + # ruby-version: 3.0 + + # # Release + # - uses: rubygems/release-gem@v1 From 47a596525294dd2457cb4e8c12ceb14009832d43 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 13:41:32 +0200 Subject: [PATCH 11/12] Test failure of github actions --- carpentry.gemspec | 1 - 1 file changed, 1 deletion(-) diff --git a/carpentry.gemspec b/carpentry.gemspec index 1ada44f..b2bd5b3 100644 --- a/carpentry.gemspec +++ b/carpentry.gemspec @@ -29,6 +29,5 @@ Gem::Specification.new do |s| if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('3.4') s.add_development_dependency "bigdecimal" s.add_development_dependency "mutex_m" - s.add_development_dependency "drb" end end From 3d74d092dd90053192552e4da3f7f6b01f628e66 Mon Sep 17 00:00:00 2001 From: Tiina Pennala Date: Wed, 22 Jan 2025 13:42:22 +0200 Subject: [PATCH 12/12] Revert "Test failure of github actions" This reverts commit 47a596525294dd2457cb4e8c12ceb14009832d43. --- carpentry.gemspec | 1 + 1 file changed, 1 insertion(+) diff --git a/carpentry.gemspec b/carpentry.gemspec index b2bd5b3..1ada44f 100644 --- a/carpentry.gemspec +++ b/carpentry.gemspec @@ -29,5 +29,6 @@ Gem::Specification.new do |s| if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('3.4') s.add_development_dependency "bigdecimal" s.add_development_dependency "mutex_m" + s.add_development_dependency "drb" end end