make_taggable 0.6.6 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 492754c144e670799533ba98bf6870d05451845d02ee347b0cca8ee8c4f7bc36
-  data.tar.gz: 2641b641d09c7a99ef706f6b446d568ddc1d9de24d9a7d1c9f007dafb5079288
+  metadata.gz: 511214c161094fadf92adb61f5735dc86af08370b92d492634170494fc5d4d8a
+  data.tar.gz: c20794c8e81c9eba4163763b86aaf1545022947d7a863f99bd3f14c8a2b6d0df
 SHA512:
-  metadata.gz: 24385c88d803dd0b4d222b998b14f6b7f9e4f2f7e84b59dd7eacf0a3ba0739e35eacda6c9ef4238b27da342c12dc75ce42f714f1ed701f3f2a70d28d4ea0d7ed
-  data.tar.gz: bf6030ccd267d19db82a6b08072ded0740f6e9674faa05cb974053289438bcc144f7da6ac97ff875fc5be8f6f585d289739ae2a80f63250fefec7eb5f658b826
+  metadata.gz: 0e10cad86c423641043013316807a1cf62eedb55be1521425bfd3d7cb235c76bf89bf832e07f520982263578c5a095ecf36f8817a5d1847df71b06f1185a98e7
+  data.tar.gz: 0d58d6f106d4031dcded133e0716577649f51c975d75675a8dc357833946d282fff3776932fe4626714fc2c92c29d213c0dd5eb361f524efef9f6a45c020b9b6

data/.github/workflows/mysql_tests.yml

@@ -0,0 +1,56 @@
+---
+name: MySQL Tests
+
+on:
+  pull_request:
+    branches:
+      - '*'
+  push:
+    branches:
+      - master
+jobs:
+  mysql:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: ['2.5', '2.6', '2.7']
+
+    services:
+      mysql:
+        image: mysql:8
+        env:
+          MYSQL_ROOT_PASSWORD: password
+          MSQL_DATABASE: my_sql_test_db
+        ports: ['3306:3306']
+        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=3
+
+    steps:
+      - uses: actions/checkout@master
+
+      - name: Set up Ruby ${{ matrix.ruby }}
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+
+      - name: Cache gems
+        uses: actions/cache@v2
+        with:
+          path: vendor/bundle
+          key: ${{ runner.os }}-${{ matrix.ruby }}-gem-${{ hashFiles('**/Gemfile.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-${{ matrix.ruby }}-gem-
+
+      - name: Install dependencies
+        run: |
+          gem install bundler
+          bundle install --jobs 4 --retry 3
+          bundle exec appraisal install
+
+      - name: Run tests
+        env:
+          DATABASE_URL: mysql2://root:password@127.0.0.1:3306/my_sql_test_db
+          RAILS_ENV: test
+        run: |
+          cp spec/dummy/config/mysql_database.yml.ci spec/dummy/config/database.yml
+          bundle exec rake test_app
+          bundle exec appraisal rake

data/.github/workflows/pg_tests.yml

@@ -0,0 +1,56 @@
+---
+name: Postgres Tests
+
+on:
+  pull_request:
+    branches:
+      - '*'
+  push:
+    branches:
+      - master
+jobs:
+  postgres:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: ['2.5', '2.6', '2.7']
+
+    services:
+      postgres:
+        image: postgres:12
+        env:
+          POSTGRES_USER: postgres
+          POSTGRES_PASSWORD: password
+          POSTGRES_DB: pg_test
+        ports: ['5432:5432']
+
+    steps:
+      - uses: actions/checkout@master
+
+      - name: Set up Ruby ${{ matrix.ruby }}
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+
+      - name: Cache gems
+        uses: actions/cache@v2
+        with:
+          path: vendor/bundle
+          key: ${{ runner.os }}-${{ matrix.ruby }}-gem-${{ hashFiles('**/Gemfile.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-${{ matrix.ruby }}-gem-
+
+      - name: Install dependencies
+        run: |
+          gem install bundler
+          bundle install --jobs 4 --retry 3
+          bundle exec appraisal install
+
+      - name: Run tests
+        env:
+          DATABASE_URL: postgres://postgres:password@localhost:5432/pg_test
+          RAILS_ENV: test
+        run: |
+          cp spec/dummy/config/pg_database.yml.ci spec/dummy/config/database.yml
+          bundle exec rake test_app
+          bundle exec appraisal rake

data/.github/workflows/{ci.yml → sqlite_tests.yml}

@@ -1,4 +1,5 @@
-name: Tests
+---
+name: SQLite Tests
 
 on:
   pull_request:
@@ -8,26 +9,27 @@ on:
     branches:
       - master
 jobs:
-  test:
+  sqlite:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ruby: [ '2.7', '2.6', '2.5' ]
+        ruby: ['2.5', '2.6', '2.7']
     steps:
       - uses: actions/checkout@master
 
-      - name: Set up Ruby
+      - name: Set up Ruby ${{ matrix.ruby }}
         uses: ruby/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby }}
 
       - name: Cache gems
-        uses: actions/cache@v1
+        uses: actions/cache@v2
         with:
           path: vendor/bundle
-          key: ${{ runner.os }}-gem-${{ hashFiles('**/Gemfile.lock') }}
+          key: ${{ runner.os }}-${{ matrix.ruby }}-gem-${{ hashFiles('**/Gemfile.lock') }}
           restore-keys: |
-            ${{ runner.os }}-gem-
+            ${{ runner.os }}-${{ matrix.ruby }}-gem-
+
       - name: Install dependencies
         run: |
           sudo apt-get update
@@ -37,11 +39,9 @@ jobs:
           bundle install --jobs 4 --retry 3
           bundle exec appraisal install
 
-      - name: StandardRb check
-        run: bundle exec standardrb
-
       - name: Run tests
         env:
           RAILS_ENV: test
         run: |
+          bundle exec rake test_app
           bundle exec appraisal rake

data/.github/workflows/standardrb-check.yml

@@ -0,0 +1,37 @@
+name: Standard Rb
+
+on:
+  pull_request:
+    branches:
+      - '*'
+  push:
+    branches:
+      - master
+
+jobs:
+  standard:
+    name: StandardRB Check Action
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@master
+    
+    - name: Set up Ruby 2.7
+      uses: actions/setup-ruby@v1
+      with:
+        ruby-version: 2.7.x
+    
+    - uses: actions/cache@v1
+      with:
+        path: vendor/bundle
+        key: ${{ runner.os }}-gems-${{ hashFiles('**/Gemfile.lock') }}
+        restore-keys: |
+          ${{ runner.os }}-gems-
+    
+    - name: Bundle install
+      run: |
+        gem install bundler
+        bundle config path vendor/bundle
+        bundle install --jobs 4 --retry 3
+    
+    - name: Run StandardRB
+      run: bundle exec standardrb --format progress

data/.gitignore

@@ -11,3 +11,6 @@ tmp
 *.lock
 *.iml
 /.idea
+spec/dummy/db/schema.rb
+spec/dummy/db/migrate/[!0]*
+/gemfiles/*

data/Appraisals

@@ -6,6 +6,10 @@ appraise "rails-6" do
   gem "rails", "~> 6.0.0"
 end
 
+appraise "rails-6-1" do
+  gem "rails", "~> 6.1.0"
+end
+
 appraise "rails-master" do
   gem "rails", github: "rails/rails"
 end

data/README.md

@@ -1,13 +1,13 @@
 # MakeTaggable
 [![Gem Version](https://badge.fury.io/rb/make_taggable.svg)](https://badge.fury.io/rb/make_taggable)
-[![Build Status](https://travis-ci.com/MatthewKennedy/make_taggable.svg?branch=master)](https://travis-ci.com/MatthewKennedy/make_taggable)
-[![Ruby Style Guide](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://github.com/testdouble/standard)
+![Postgres Tests](https://github.com/MatthewKennedy/make_taggable/workflows/Postgres%20Tests/badge.svg)
+![MySQL Tests](https://github.com/MatthewKennedy/make_taggable/workflows/MySQL%20Tests/badge.svg)
+![SQLite Tests](https://github.com/MatthewKennedy/make_taggable/workflows/SQLite%20Tests/badge.svg)
+![Standard Rb](https://github.com/MatthewKennedy/make_taggable/workflows/Standard%20Rb/badge.svg)
 
-MakeTaggable is a fork of acts-as-taggable-on v6.5 with code updates and a new set of migrations. All credit goes to those who contributed before acts-as-taggable-on became MakeTaggable: Michael Bleigh & Joost Baaij.
+MakeTaggable is a fork of Acts-As-Taggable-On with code updates, fresh migrations that work on MySQL, and tested against a dummy Rails app.
 
-Why fork acts-as-taggable-on? Act As Taggable On appears stuck with a set of legacy migrations from rails pre [4.2] that added and removed indexes so much that the migrations do not run on a fresh install using MySQL.
-
-For the PostgreSQL users, this is not an issue, but if you have an app or gem that used acts-as-taggable-on, you can no longer test against MySQL, so it seems a fresh start would be useful to clear out any legacy issues and move forward.
+All credit goes to those who contributed to Acts-As-Taggable-On: Michael Bleigh & Joost Baaij.
 
 ## Installation
 
@@ -22,23 +22,22 @@ bundle add make_taggable
 Install migrations
 
 ```shell
-# For the latest versions :
 rails make_taggable_engine:install:migrations
 ```
 
-Review the generated migrations then migrate :
+Review the generated migrations then migrate:
 ```shell
 rails db:migrate
 ```
 
 #### For MySql users
-You can circumvent at any time the problem of special characters [issue 623](https://github.com/mbleigh/acts-as-taggable-on/issues/623) by setting in an initializer file:
+To make MySQL play nice with spÉcial characters you can setting the following line in an initializer file:
 
 ```ruby
 MakeTaggable.force_binary_collation = true
 ```
 
-Or by running this rake task:
+Or by run this rake task:
 
 ```shell
 rails make_taggable_engine:tag_names:collate_bin
@@ -46,6 +45,7 @@ rails make_taggable_engine:tag_names:collate_bin
 
 See the Configuration section for more details.
 
+
 ## Usage
 
 Setup
@@ -176,7 +176,7 @@ MakeTaggable::Tag.least_used(10)
 
 ### Finding Tagged Objects
 
-Make Taggable uses scopes to create an association for tags.
+MakeTaggable uses scopes to create an association for tags.
 This way you can mix and match to filter down your results.
 
 ```ruby
@@ -405,6 +405,7 @@ CSS:
 .css4 { font-size: 1.6em; }
 ```
 
+
 ## Configuration
 
 If you would like to remove unused tag objects after removing taggings, add:
@@ -453,35 +454,54 @@ MakeTaggable.delimiter = ','
 *NOTE 1: SQLite by default can't upcase or downcase multibyte characters, resulting in unwanted behavior. Load the SQLite ICU extension for proper handle of such characters. [See docs](http://www.sqlite.org/src/artifact?ci=trunk&filename=ext/icu/README.txt)*
 
 
-#### Upgrading
+## Upgrading
 
-see [UPGRADING](UPGRADING.md)
+Install new migrations by running:
 
-## Contributors
-
-We have a long list of valued contributors. [Check them all](https://github.com/mbleigh/acts-as-taggable-on/contributors)
+```shell
+rails make_taggable_engine:install:migrations
+```
 
 ## Compatibility
 
-Versions 0.6.x is compatible with Ruby 2.5 + and Rails 6.
-
+Version 0.7.x is compatible with Ruby 2.5 > and Rails 5.2 >
 
-## TODO
-- Review migrations indexes they seem excessive looking at the schema.
-- See if the newer version of ActiveRecord or arel can be used instead of the MySQL workarounds.
 
 ## Testing
 
-Make Taggable uses RSpec for its test coverage. Inside the gem
-directory, you can run the specs with:
+MakeTaggable uses RSpec for its test coverage. Inside the gem directory, you can run the specs by following the steps below:
+
+Install the rquired gems:
+
+```shell
+bundle install
+```
+
+Setup the dummy test app:
 
 ```shell
-bundle
-rake spec
+bundle exec rake test_app
 ```
 
-You can run all the tests across all the Rails versions by running `rake appraise`.  If you'd also like to [run the tests across all rubies and databases as configured for Travis CI, install and run `wwtd`](https://github.com/grosser/wwtd).
+Run the spec tests against SQLite use the followiong command.
+
+```shell
+bundle exec rake
+```
+
+You can also run all the tests across all the Rails versions by running:
+
+```shell
+bundle exec appraisal install
+
+bundle exec appraisal rake
+```
+
+
+## Contributors
+
+List of valued contributors From Acts-As-Taggable-On: [Check them all](https://github.com/mbleigh/acts-as-taggable-on/contributors)
 
 
 ## License
-MIT
+This gem is available as open source under the terms of the [MIT License](https://github.com/MatthewKennedy/make_taggable/blob/master/LICENSE.txt).

data/Rakefile

@@ -8,3 +8,4 @@ require "rspec/core/rake_task"
 RSpec::Core::RakeTask.new(:spec)
 task default: :spec
 
+load "lib/tasks/setup_test_db.rake"

data/lib/make_taggable.rb

@@ -111,10 +111,10 @@ module MakeTaggable
 
     def self.apply_binary_collation(bincoll)
       if Utils.using_mysql?
-        coll = "utf8_general_ci"
-        coll = "utf8_bin" if bincoll
+        coll = "utf8mb4_general_ci"
+        coll = "utf8mb4_bin" if bincoll
         begin
-          ActiveRecord::Migration.execute("ALTER TABLE #{Tag.table_name} MODIFY name varchar(255) CHARACTER SET utf8 COLLATE #{coll};")
+          ActiveRecord::Migration.execute("ALTER TABLE #{Tag.table_name} MODIFY name varchar(255) CHARACTER SET utf8mb4 COLLATE #{coll};")
         rescue => e
           puts "Trapping #{e.class}: collation parameter ignored while migrating for the first time."
         end

data/lib/make_taggable/version.rb

@@ -1,4 +1,3 @@
-# Acts As Taggable With Updates
 module MakeTaggable
-  VERSION = "0.6.6"
+  VERSION = "0.7.0"
 end

data/lib/tasks/setup_test_db.rake

@@ -0,0 +1,6 @@
+require "fileutils"
+
+desc "Set up the test app for"
+task :test_app do
+  exec "cd spec/dummy && rake make_taggable_engine:install:migrations && rake db:create RAILS_ENV=test && rake db:migrate RAILS_ENV=test"
+end

data/make_taggable.gemspec

@@ -6,25 +6,27 @@ Gem::Specification.new do |spec|
   spec.name = "make_taggable"
   spec.version = MakeTaggable::VERSION
   spec.authors = ["Matthew Kennedy", "Michael Bleigh", "Joost Baaij"]
-  spec.email = %w[m.kennedy@me.com michael@intridea.com joost@spacebabies.nl]
-  spec.description = "A fork of acts-as-taggable-on v6.5 with code updates and a new set of migrations."
+  spec.email = %w[m.kennedy@me.com]
+  spec.description = "MakeTaggable is a fork of Acts-As-Taggable-On with code updates & fresh migrations"
   spec.summary = "Advanced Tagging For Rails"
   spec.homepage = "https://github.com/MatthewKennedy/make_taggable"
   spec.license = "MIT"
 
-  if File.exist?('UPGRADING.md')
-    spec.post_install_message = File.read('UPGRADING.md')
+  if File.exist?("UPGRADING.md")
+    spec.post_install_message = File.read("UPGRADING.md")
   end
 
   spec.files = `git ls-files`.split($/)
   spec.test_files = spec.files.grep(%r{^spec/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rails", ">= 5.2"
+  spec.add_dependency "rails"
 
-  spec.add_development_dependency "rspec", ">=3.0"
+  spec.add_development_dependency "appraisal"
+  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"
   spec.add_development_dependency "standard"
-  spec.add_development_dependency "appraisal"
+  spec.add_development_dependency "sqlite3"
 end