order_as_specified 1.6 → 1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 119668cc688124610b87036f0f929c0fbcc35ef4acea5e1c2a1cdc48c1101241
-  data.tar.gz: a17df8766bfe178cac883311d8ba4b165b4e43a9c4afd1d8e4b8af0e099d5010
+  metadata.gz: d928c44adb61e277076bd76af0ac1568b132a0f9ab49050d1b047fee59305714
+  data.tar.gz: bcbf4c9b1f4061ae9798535dff306c27c7b845bd827a9cd9f2e0ee7ee195c427
 SHA512:
-  metadata.gz: 1a53e19dc18fb420fbf88f1fb535b46a0a5c5b13a84158c237a350243f8cfefa5f78ae048087bcc939ea7ecab12e4b7aec5c62bae7960d921a83704f9a53a45e
-  data.tar.gz: 0ca1d4697d3634f442edb58173cf4d7f6199571ae1a69415cb6cb2d8b93079ff588023e5549f99ca998979cfeff93ee3ea1f2f0810afd90c8501df829d41e104
+  metadata.gz: 58d477da354092f0f3c7e9a034c14a76b04ac80599d46850ce064dd84ce0d172cbef0efd9ac6a3a7c467ba7472250299698aad1aa6311f7f679a4cc44d231483
+  data.tar.gz: bcf2ab1b90641f027ea0b9200bc442d77902448764a4f473552d2f6d4ec46c30023ce7bf893ddd678787212858ee2a85a738c0d10008d85fd7911fc3a37f2b17

data/.dependabot/config.yml

@@ -0,0 +1,46 @@
+# This is a dependabot configuration file. When this file is seen on github, a
+# dependabot configuration is created for the project. We can use this to
+# control various aspects of the automated dependency checking, such as the
+# frequency and the target_branch.
+#
+# Reference: https://dependabot.com/docs/config-file/
+version: 1
+update_configs:
+  # This configures dependency updates for one package manager. In some
+  # projects, such as warehouse, where we have Ruby and Python, there can be
+  # separate package_manager entries.
+  - package_manager: "ruby:bundler"
+    directory: "/"
+    update_schedule: "weekly"
+
+    default_labels:
+      - "dependencies"
+      - "Needs QA"
+
+    # Dependabot will use a repository's default branch. This will override
+    # that.
+    # target_branch: "master"
+
+    allowed_updates:
+      - match:
+          dependency_type: "direct"
+
+    automerged_updates:
+      # This allows all dependencies that are used for development, e.g., rspec,
+      # rspec-mock, vcr, etc, to be automatically updated. This is generally
+      # okay because the dependencies are not used in production.
+      - match:
+          dependency_type: "development"
+          update_type: "all"
+
+    #   # This is an example entry to enable automerging of a specific dependency
+    #   # when the update is only for minor or patch semantic versions.
+    #   #
+    #   # The dependency_name can also be a wildcard.
+    #   #
+    #   # This is left commented, but whitelisting a dependency for automatic
+    #   # merging is as simple as creating a new entry that looks like the below.
+    #   - match:
+    #       dependency_type: "all"
+    #       dependency_name: "aws-sdk-s3"
+    #       update_type: "semver:minor"

data/.github/workflows/auto-approve-dependabot.yml

@@ -0,0 +1,26 @@
+# This workflow auto-approves pull-requests when the github actor is a
+# dependabot user and it is opening a new pull request.
+#
+# The problem that this workflow solves is that we have branch protection on
+# our repositories that prevent PRs from merging unless there is an
+# approval present. The problem is that this will block PRs that dependabot
+# may want to merge automatically. Auto-approving dependabot PRs will allow
+# the automatic merge to complete. We control what gets automerged through
+# the dependabot configuration.
+#
+# This is a known issue: https://github.com/dependabot/feedback/issues/852
+name: Auto-approve dependabot pull requests
+on:
+  pull_request:
+    types: [opened]
+
+jobs:
+  dependabot-triage:
+    runs-on: ubuntu-latest
+    if: (github.actor == 'dependabot[bot]' || github.actor == 'dependabot-preview[bot]')
+
+    steps:
+      - name: Auto-approve for dependabot
+        uses: hmarr/auto-approve-action@v2.0.0
+        with:
+          github-token: "${{ secrets.GITHUB_TOKEN }}"

data/.github/workflows/remove-needs-qa.yml

@@ -0,0 +1,35 @@
+# This workflow removes a "Needs QA" label from a PR when the actor is the
+# dependabot user merging a PR.
+#
+# We need this mechanism to allow for automerging whitelisted dependencies while
+# also allowing for blocking a merge to master for deployment (in the way that
+# our other PRs work). When the automerge script runs in henchman, it looks
+# for `Needs QA` on github pull requests, and if the label is present,
+# blocks the commit from merging.
+name: Remove 'Needs QA' label for auto-merged PRs.
+on:
+  pull_request:
+    types: [closed]
+
+jobs:
+  remove-label:
+    runs-on: ubuntu-latest
+    if: >
+      (github.actor == 'dependabot[bot]' || github.actor == 'dependabot-preview[bot]')
+      && github.event.pull_request.merged
+
+    steps:
+      # Our triage workflow adds 'Needs QA' to the PR in order to block it from
+      # merging to production. This removes that label when dependabot is doing
+      # the merging.
+      - name: Remove QA Label
+        uses: actions/github-script@0.4.0
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            github.issues.removeLabel({
+              issue_number: context.issue.number,
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              name: 'Needs QA'
+            })

data/.github/workflows/tests.yml

@@ -0,0 +1,61 @@
+# based on https://github.com/ruby/setup-ruby/blob/master/README.md
+name: Tests
+on: [push, pull_request]
+jobs:
+  ci:
+    name: CI
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ ubuntu-latest ]
+        ruby: [ 2.5, 2.6, 2.7 ]
+    runs-on: ${{ matrix.os }}
+    services:
+      postgres:
+        image: postgres:latest
+        env:
+          POSTGRES_USER: postgres
+          POSTGRES_PASSWORD: postgres
+          POSTGRES_DB: order_as_specified_test
+        options: >-
+          --health-cmd pg_isready
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+        ports:
+          - 5432:5432
+      mysql:
+        image: mysql:latest
+        env:
+            MYSQL_USER: mysql
+            MYSQL_PASSWORD: mysql
+            MYSQL_ROOT_PASSWORD: mysql
+            MYSQL_DATABASE: order_as_specified_test
+        ports:
+            - 3306:3306
+        options: >-
+          --health-cmd "mysqladmin ping"
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+    env:
+      CI: true
+    steps:
+    - uses: actions/checkout@v2
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+    - uses: actions/cache@v1
+      with:
+        path: vendor/bundle
+        key: bundle-use-ruby-${{ matrix.os }}-${{ matrix.ruby }}-${{ hashFiles('**/Gemfile.lock') }}
+        restore-keys: |
+          bundle-use-ruby-${{ matrix.os }}-${{ matrix.ruby }}-
+    - run: sudo apt-get install libsqlite3-dev
+    - name: bundle install
+      run: |
+        ruby -v
+        bundle config path vendor/bundle
+        bundle install --jobs 4 --retry 3
+    - run: bundle exec rubocop
+    - run: bundle exec rspec

data/.gitignore

@@ -13,4 +13,3 @@
 *.a
 mkmf.log
 *.gem
-.rubocop-https---raw-githubusercontent-com-panorama-ed-code-conventions-master-rubocop-yml

data/.rubocop.yml

@@ -1,5 +1,2 @@
-inherit_from:
-  - https://raw.githubusercontent.com/panorama-ed/code-conventions/master/rubocop.yml
-
-AllCops:
-  TargetRubyVersion: 2.4 # Keep this in sync with .travis.yml
+inherit_gem:
+  panolint: rubocop.yml

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Unreleased (`master`)
 
+# 1.7
+
+This release adds support for `nil` values in the ordering list. Thanks to
+[yfulmes](https://github.com/yfulmes) for requesting this feature!
+
 # 1.6
 
 We are dropping official support for Ruby 2.3 and below, though they may

data/CODE_OF_CONDUCT.md

@@ -0,0 +1,74 @@
+# Contributor Covenant Code of Conduct
+
+## Our Pledge
+
+In the interest of fostering an open and welcoming environment, we as
+contributors and maintainers pledge to making participation in our project and
+our community a harassment-free experience for everyone, regardless of age, body
+size, disability, ethnicity, gender identity and expression, level of experience,
+nationality, personal appearance, race, religion, or sexual identity and
+orientation.
+
+## Our Standards
+
+Examples of behavior that contributes to creating a positive environment
+include:
+
+* Using welcoming and inclusive language
+* Being respectful of differing viewpoints and experiences
+* Gracefully accepting constructive criticism
+* Focusing on what is best for the community
+* Showing empathy towards other community members
+
+Examples of unacceptable behavior by participants include:
+
+* The use of sexualized language or imagery and unwelcome sexual attention or
+advances
+* Trolling, insulting/derogatory comments, and personal or political attacks
+* Public or private harassment
+* Publishing others' private information, such as a physical or electronic
+  address, without explicit permission
+* Other conduct which could reasonably be considered inappropriate in a
+  professional setting
+
+## Our Responsibilities
+
+Project maintainers are responsible for clarifying the standards of acceptable
+behavior and are expected to take appropriate and fair corrective action in
+response to any instances of unacceptable behavior.
+
+Project maintainers have the right and responsibility to remove, edit, or
+reject comments, commits, code, wiki edits, issues, and other contributions
+that are not aligned to this Code of Conduct, or to ban temporarily or
+permanently any contributor for other behaviors that they deem inappropriate,
+threatening, offensive, or harmful.
+
+## Scope
+
+This Code of Conduct applies both within project spaces and in public spaces
+when an individual is representing the project or its community. Examples of
+representing a project or community include using an official project e-mail
+address, posting via an official social media account, or acting as an appointed
+representative at an online or offline event. Representation of a project may be
+further defined and clarified by project maintainers.
+
+## Enforcement
+
+Instances of abusive, harassing, or otherwise unacceptable behavior may be
+reported by contacting the project team at engineering@panoramaed.com. All
+complaints will be reviewed and investigated and will result in a response that
+is deemed necessary and appropriate to the circumstances. The project team is
+obligated to maintain confidentiality with regard to the reporter of an incident.
+Further details of specific enforcement policies may be posted separately.
+
+Project maintainers who do not follow or enforce the Code of Conduct in good
+faith may face temporary or permanent repercussions as determined by other
+members of the project's leadership.
+
+## Attribution
+
+This Code of Conduct is adapted from the [Contributor Covenant][homepage], version 1.4,
+available at [https://contributor-covenant.org/version/1/4][version]
+
+[homepage]: https://contributor-covenant.org
+[version]: https://contributor-covenant.org/version/1/4/

data/Gemfile

@@ -2,9 +2,11 @@
 
 source "https://rubygems.org"
 
-# Specify your gem's dependencies in order_as_specified.gemspec
+git_source(:github) { |repo| "https://github.com/#{repo}.git" }
+
+# Specify your gem's dependencies in unique_attributes.gemspec
 gemspec
 
-if ENV["TRAVIS"] == "true" && ENV["ACTIVERECORD_VERSION"]
-  gem "activerecord", ENV["ACTIVERECORD_VERSION"]
+group :development do
+  gem "panolint", github: "panorama-ed/panolint", branch: "main"
 end

data/README.md

@@ -142,6 +142,20 @@ TestObject.order_as_specified(language: ["fr", "es"])
    ]>
 ```
 
+The order can also include `nil` attributes:
+
+```ruby
+TestObject.order_as_specified(language: ["es", nil, "fr"])
+=> #<ActiveRecord::Relation [
+     #<TestObject id: 3, language: "es">,
+     #<TestObject id: 1, language: nil>,
+     #<TestObject id: 4, language: nil>,
+     #<TestObject id: 2, language: "fr">
+   ]>
+```
+
+### `distinct_on`
+
 In databases that support it (such as PostgreSQL), you can also use an option to
 add a `DISTINCT ON` to your query when you would otherwise have duplicates:
 
@@ -154,10 +168,20 @@ TestObject.order_as_specified(distinct_on: true, language: ["fr", "en"])
    ]>
 ```
 
-Note that if a `nil` value is passed in the ordering an error is raised, because
-databases do not have good or consistent support for ordering with `NULL` values
-in an arbitrary order, so we don't permit this behavior instead of allowing an
-unexpected result.
+### `case_insensitive`
+
+If you want objects to come back in an order that is case-insensitive, you can
+pass the `case_insensitive: true` value to the `order_as_specified` call, as in:
+
+```ruby
+TestObject.order_as_specified(case_insensitive: true, language: ["fr", "en"])
+=> #<ActiveRecord::Relation [
+     #<TestObject language: "fr">
+     #<TestObject language: "FR">
+     #<TestObject language: "EN">
+     #<TestObject language: "en">
+   ]>
+```
 
 ## Limitations
 

data/RELEASING.md

@@ -6,9 +6,9 @@
 1. On the new branch, update the VERSION constant in `lib/order_as_specified/version.rb`.
 1. Update the Changelog.
 1. Commit the change: `git add -A && git commit -m 'Bump to vX.X'`.
-1. Add a tag: `git tag -am "vX.X" vX.X`.
-1. Push the branch and tag: `git push --follow-tags`
 1. Make a PR.
 1. Merge the PR.
+1. Add a tag: `git tag -am "vX.X" vX.X`.
+1. Push the tag: `git push --tags`
 1. Push to rubygems: `gem build order_as_specified.gemspec && gem push *.gem && rm *.gem`
 1. Celebrate!

data/lib/order_as_specified.rb

@@ -12,33 +12,39 @@ module OrderAsSpecified
   # @return [ActiveRecord::Relation] the objects, ordered as specified
   def order_as_specified(hash)
     distinct_on = hash.delete(:distinct_on)
+    case_insensitive = hash.delete(:case_insensitive)
+
     params = extract_params(hash)
     return all if params[:values].empty?
 
-    table = connection.quote_table_name(params[:table])
-    attribute = connection.quote_column_name(params[:attribute])
+    table = Arel::Table.new(params[:table])
+    node = Arel::Nodes::Case.new
 
-    conditions = params[:values].map do |value|
-      raise OrderAsSpecified::Error, "Cannot order by `nil`" if value.nil?
+    params[:values].each_with_index do |value, index|
+      attribute = table[params[:attribute]]
+      condition =
+        if value.is_a?(Range)
+          if value.first >= value.last
+            raise OrderAsSpecified::Error, "Range needs to be increasing"
+          end
 
-      if value.is_a? Range
-        range_clause("#{table}.#{attribute}", value)
-      else
-        # Sanitize each value to reduce the risk of SQL injection.
-        "#{table}.#{attribute}=#{quote(value)}"
-      end
-    end
+          attribute.between(value)
+        elsif case_insensitive
+          attribute.matches(value)
+        else
+          attribute.eq(value)
+        end
 
-    when_queries = conditions.map.with_index do |cond, index|
-      "WHEN #{cond} THEN #{index}"
+      node.when(condition).then(index)
     end
-    case_query = "CASE #{when_queries.join(' ')} ELSE #{conditions.size} END"
-    scope = order(Arel.sql("#{case_query} ASC"))
+
+    node.else(node.conditions.size)
+    scope = order(Arel::Nodes::Ascending.new(table.grouping(node)))
 
     if distinct_on
-      scope = scope.select(
-        Arel.sql("DISTINCT ON (#{case_query}) #{table}.*")
-      )
+      distinct = Arel::Nodes::DistinctOn.new(node)
+      table_alias = connection.quote_table_name(table.name)
+      scope = scope.select(Arel.sql("#{distinct.to_sql} #{table_alias}.*"))
     end
 
     scope
@@ -69,20 +75,4 @@ module OrderAsSpecified
       }
     end
   end
-
-  def range_clause(col, range)
-    if range.first >= range.last
-      raise OrderAsSpecified::Error, "Range needs to be increasing"
-    end
-
-    op = range.exclude_end? ? "<" : "<="
-    "#{col} >= #{quote(range.first)} AND #{col} #{op} #{quote(range.last)}"
-  end
-
-  def quote(value)
-    # We have to explicitly quote for now because SQL sanitization for ORDER BY
-    # queries isn't in less current versions of Rails.
-    # See: https://github.com/rails/rails/pull/13008
-    ActiveRecord::Base.connection.quote(value)
-  end
 end

data/lib/order_as_specified/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OrderAsSpecified
-  VERSION = "1.6"
+  VERSION = "1.7"
 end

data/order_as_specified.gemspec

@@ -23,10 +23,13 @@ Gem::Specification.new do |spec|
   spec.add_dependency "activerecord", ">= 5.0.0"
 
   spec.add_development_dependency "bundler"
-  spec.add_development_dependency "codecov"
   spec.add_development_dependency "mysql2"
   spec.add_development_dependency "pg"
   spec.add_development_dependency "rspec"
   spec.add_development_dependency "rspec-rails"
-  spec.add_development_dependency "sqlite3", "~> 1.3.13"
+
+  # Older versions of Rails locked in the SQLite3 version, so we have to
+  # explicitly specify the version here
+  sqlite3 = ENV["ACTIVERECORD_VERSION"] == "~> 5.0.0" ? "~> 1.3.13" : "~> 1.4"
+  spec.add_development_dependency "sqlite3", sqlite3
 end

data/spec/config/database.yml

@@ -5,11 +5,15 @@ sqlite3_test:
 
 postgresql_test:
   adapter: postgresql
+  host: localhost
   database: order_as_specified_test
   username: postgres
+  password: postgres
 
 mysql_test:
   adapter: mysql2
-  database: order_as_specified_test
-  username: travis
+  host: 127.0.0.1
   encoding: utf8
+  database: order_as_specified_test
+  username: mysql
+  password: mysql

data/spec/config/test_setup_migration.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-VersionedMigration = ActiveRecord::Migration[ActiveRecord::Migration.current_version] # rubocop:disable Metrics/LineLength
+VersionedMigration = ActiveRecord::Migration[ActiveRecord::Migration.current_version] # rubocop:disable Layout/LineLength
 
 class TestSetupMigration < VersionedMigration
   def up

data/spec/mysql_spec.rb

@@ -5,12 +5,12 @@ require "shared/order_as_specified_examples"
 require "config/test_setup_migration"
 
 RSpec.describe "MySQL" do
-  before :all do
+  before :all do # rubocop:disable RSpec/BeforeAfterAll
     ActiveRecord::Base.establish_connection(:mysql_test)
     TestSetupMigration.migrate(:up)
   end
 
-  after(:all) { ActiveRecord::Base.remove_connection }
+  after(:all) { ActiveRecord::Base.remove_connection } # rubocop:disable RSpec/BeforeAfterAll
 
   include_examples ".order_as_specified"
 end

data/spec/postgresql_spec.rb

@@ -5,12 +5,12 @@ require "shared/order_as_specified_examples"
 require "config/test_setup_migration"
 
 RSpec.describe "PostgreSQL" do
-  before :all do
+  before :all do # rubocop:disable RSpec/BeforeAfterAll
     ActiveRecord::Base.establish_connection(:postgresql_test)
     TestSetupMigration.migrate(:up)
   end
 
-  after(:all) { ActiveRecord::Base.remove_connection }
+  after(:all) { ActiveRecord::Base.remove_connection } # rubocop:disable RSpec/BeforeAfterAll
 
   include_examples ".order_as_specified"
 
@@ -31,7 +31,7 @@ RSpec.describe "PostgreSQL" do
       expect(subject.length).to eq 3
     end
 
-    context "input safety" do
+    describe "input safety" do
       before(:each) { TestClass.create(field: "foo") }
 
       it "sanitizes column values" do
@@ -61,8 +61,9 @@ RSpec.describe "PostgreSQL" do
           distinct_on: true,
           table => { column => ["foo"] }
         ).to_sql
-        pattern = "DISTINCT ON (CASE WHEN #{quoted_table}.#{quoted_column}"
-        expect(sql).to include(pattern)
+
+        pattern = /DISTINCT ON \(\s*CASE WHEN #{quoted_table}.#{quoted_column}/
+        expect(sql).to match(pattern)
       end
     end
   end

data/spec/shared/order_as_specified_examples.rb

@@ -58,8 +58,8 @@ RSpec.shared_examples ".order_as_specified" do
         end.shuffle
       end
 
-      it "raises an error" do
-        expect { subject }.to raise_error(OrderAsSpecified::Error)
+      it "returns results in the given order" do
+        expect(subject.map(&:id)).to eq shuffled_object_ids
       end
     end
   end
@@ -101,7 +101,7 @@ RSpec.shared_examples ".order_as_specified" do
       ]
     end
 
-    context "exclusive ranges" do
+    context "when ranges are exclusive" do
       let(:numbers) { [0, 1, 2, 3, 4, 0.9, 1.5] }
 
       let(:ranges) { [(1...2), (0...1), (2...5)] }
@@ -119,7 +119,7 @@ RSpec.shared_examples ".order_as_specified" do
       end
     end
 
-    context "reverse ranges" do
+    context "when ranges are in reverse order" do
       let(:ranges) { [(5..0)] }
 
       it "raises an error" do
@@ -163,7 +163,7 @@ RSpec.shared_examples ".order_as_specified" do
     end
   end
 
-  context "input safety" do
+  describe "input safety" do
     before(:each) do
       2.times { |i| TestClass.create(field: "foo#{i}") }
     end
@@ -192,16 +192,39 @@ RSpec.shared_examples ".order_as_specified" do
       quoted_column = AssociationTestClass.connection.quote_column_name(column)
 
       sql = TestClass.order_as_specified(table => { column => ["foo"] }).to_sql
-      pattern = "ORDER BY CASE WHEN #{quoted_table}.#{quoted_column}"
+      pattern = "ORDER BY (CASE WHEN #{quoted_table}.#{quoted_column}"
       expect(sql).to include(pattern)
     end
   end
 
-  context "invalid hash input" do
+  context "when hash input is invalid" do
     subject { TestClass.order_as_specified({}) }
 
     it "raises an error" do
       expect { subject }.to raise_error(OrderAsSpecified::Error)
     end
   end
+
+  context "when case insensitive option is used" do
+    subject do
+      TestClass.
+        order_as_specified(field: %w[abc def], case_insensitive: true).
+        pluck(TestClass.arel_table[:field].lower)
+    end
+
+    before :each do
+      TestClass.create!(
+        [
+          { field: "dEf" },
+          { field: "aBc" },
+          { field: "ABC" },
+          { field: "DEF" }
+        ]
+      )
+    end
+
+    it "orders in a case insensitive manner" do
+      expect(subject).to eq(%w[abc abc def def])
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -1,15 +1,5 @@
 # frozen_string_literal: true
 
-if ENV["TRAVIS"] == "true" && ENV["CODE_COVERAGE"] == "true"
-  require "simplecov"
-  require "codecov"
-  SimpleCov.formatter = SimpleCov::Formatter::Codecov
-  SimpleCov.start do
-    # Omit the spec directory from being counted in code coverage calculations.
-    add_filter "/spec/"
-  end
-end
-
 require "active_record"
 
 require "order_as_specified"

data/spec/sqlite3_spec.rb

@@ -5,12 +5,12 @@ require "shared/order_as_specified_examples"
 require "config/test_setup_migration"
 
 RSpec.describe "SQLite3" do
-  before :all do
+  before :all do # rubocop:disable RSpec/BeforeAfterAll
     ActiveRecord::Base.establish_connection(:sqlite3_test)
     TestSetupMigration.migrate(:up)
   end
 
-  after(:all) { ActiveRecord::Base.remove_connection }
+  after(:all) { ActiveRecord::Base.remove_connection } # rubocop:disable RSpec/BeforeAfterAll
 
   include_examples ".order_as_specified"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: order_as_specified
 version: !ruby/object:Gem::Version
-  version: '1.6'
+  version: '1.7'
 platform: ruby
 authors:
 - Jacob Evelyn
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-02-11 00:00:00.000000000 Z
+date: 2021-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -38,20 +38,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: codecov
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: mysql2
   requirement: !ruby/object:Gem::Requirement
@@ -114,14 +100,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.13
+        version: '1.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.13
+        version: '1.4'
 description: Obtain ActiveRecord results with a custom ordering with no need to store
   anything in the database.
 email:
@@ -130,10 +116,14 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".dependabot/config.yml"
+- ".github/workflows/auto-approve-dependabot.yml"
+- ".github/workflows/remove-needs-qa.yml"
+- ".github/workflows/tests.yml"
 - ".gitignore"
 - ".rubocop.yml"
-- ".travis.yml"
 - CHANGELOG.md
+- CODE_OF_CONDUCT.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -157,7 +147,7 @@ homepage: https://github.com/panorama-ed/order_as_specified
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -172,9 +162,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
-signing_key: 
+rubyforge_project:
+rubygems_version: 2.7.6.2
+signing_key:
 specification_version: 4
 summary: Add arbitrary ordering to ActiveRecord queries.
 test_files:

data/.travis.yml

@@ -1,41 +0,0 @@
-language: ruby
-rvm:
-- 2.6
-- 2.5
-- 2.4 # Keep this in sync with .rubocop.yml
-# We test up to the latest version of Ruby here, and test these Ruby versions
-# against all support ActiveRecord versions. One test will be "duplicated"
-# below when we calculate code coverage, but I don't know how to avoid that and
-# still test all combinations of Ruby and ActiveRecord.
-before_script:
-- psql -c 'create database order_as_specified_test;' -U postgres
-- mysql -e 'CREATE DATABASE order_as_specified_test;'
-script:
-- bundle exec rspec
-services:
-- mysql
-- postgresql
-env:
-  matrix:
-  - ACTIVERECORD_VERSION="~> 5.2.0"
-  - ACTIVERECORD_VERSION="~> 5.1.0"
-  - ACTIVERECORD_VERSION="~> 5.0.0"
-matrix:
-  include:
-  - rvm: 2.6
-    script:
-    - bundle exec rspec
-    - gem install --no-document rubocop rubocop-rspec-focused && rubocop
-    env:
-    - CODE_COVERAGE=true
-    - ACTIVERECORD_VERSION="~> 5.2.0"
-branches:
-  only:
-  # We always run tests for PRs. This prevents running PR tests twice (once "for
-  # the PR" and once "for the branch"), though we do want tests to run always
-  # run on master.
-  - master
-notifications:
-  email: false
-  slack:
-    secure: lVaScxPymqeYlTh+KlprWDK+O18BkQTyNSBTfzsLFzWF3Dy+p1rp2aeh0AhuLVYQDrlmFZ7cDMZ1ZYDyV2Y6lLtt+4ii7rATagI8R9WUVYb90C97tu1M6v5tpIjoTEOLx9hGc66heyvq1hNCKXbTqsqmrD69FYdjuXnOfrIs6J4=