Merge pull request #220 from ydah/fix-typos

Add CodeSpell workflow for spell checking in pull requests

Author: krororo

.codespellignore

@@ -0,0 +1,3 @@
+COO
+NotIn
+xdescribe

.github/workflows/codespell.yml

@@ -0,0 +1,15 @@
+name: CodeSpell
+on:
+  - pull_request
+jobs:
+  codespell:
+    name: CodeSpell
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+      - name: CodeSpell
+        uses: codespell-project/actions-codespell@406322ec52dd7b488e48c1c4b82e2a8b3a1bf630 # v2.1
+        with:
+          check_filenames: true
+          check_hidden: true
+          ignore_words_file: .codespellignore

CHANGELOG.md

@@ -26,7 +26,7 @@
 
 - [Add a note to the README that PostgreSQL is required to run the tests. #188](https://github.com/kufu/activerecord-bitemporal/pull/188)
 - [Remove specs for Rails 5.x #191](https://github.com/kufu/activerecord-bitemporal/pull/191)
-- [Update auto assgin member #193](https://github.com/kufu/activerecord-bitemporal/pull/193)
+- [Update auto assign member #193](https://github.com/kufu/activerecord-bitemporal/pull/193)
 - [Pin GitHub Actions dependencies to specific commit hashes #194](https://github.com/kufu/activerecord-bitemporal/pull/194)
 - [Bump ruby/setup-ruby from 1.227.0 to 1.229.0 #195](https://github.com/kufu/activerecord-bitemporal/pull/195)
 - [Bump ruby/setup-ruby from 1.229.0 to 1.233.0 #197](https://github.com/kufu/activerecord-bitemporal/pull/197)
@@ -35,7 +35,7 @@
 - [Bump ruby/setup-ruby from 1.237.0 to 1.238.0 #201](https://github.com/kufu/activerecord-bitemporal/pull/201)
 - [Bump ruby/setup-ruby from 1.238.0 to 1.242.0 #202](https://github.com/kufu/activerecord-bitemporal/pull/202)
 - [Bump ruby/setup-ruby from 1.242.0 to 1.244.0 #203](https://github.com/kufu/activerecord-bitemporal/pull/203)
-- [Update auto assgin member #204](https://github.com/kufu/activerecord-bitemporal/pull/204)
+- [Update auto assign member #204](https://github.com/kufu/activerecord-bitemporal/pull/204)
 
 ## 5.3.0
 

spec/activerecord-bitemporal/bitemporal_scope_spec.rb

@@ -974,7 +974,7 @@ class Article < ActiveRecord::Base
         it { is_expected.to have_bitemporal_at(time_current, table: "blogs") }
         it { is_expected.to have_bitemporal_at(time_current, table: "articles") }
 
-        # Not suppoted call to_sql outside unscoped
+        # Not supported call to_sql outside unscoped
         xcontext "with call to_sql outside `Article.unscoped`" do
           let(:sql) { Article.unscoped { Blog.joins(:articles) }.to_sql }
           it { is_expected.to have_bitemporal_at(time_current, table: "blogs") }
@@ -1024,7 +1024,7 @@ class Article < ActiveRecord::Base
             let(:sql) { Article.unscoped { Blog.joins(:articles).merge(Article.valid_at("2019/01/01")).to_sql } }
             it { is_expected.to have_bitemporal_at(time_current, table: "blogs") }
             it { is_expected.to have_valid_at("2019/01/01".in_time_zone, table: "articles") }
-            # Not suppoted merge
+            # Not supported merge
             # `Article.valid_at("2019/01/01")` without `default_scope` in `Article.unscoped`
             xit { is_expected.to have_transaction_at(time_current, table: "articles") }
 

spec/activerecord-bitemporal/bitemporal_spec.rb

@@ -1181,7 +1181,7 @@ class EmployeeWithUniquness < Employee
         it_behaves_like "return false and #destroyed? to be false"
       end
 
-      context "with `destory!`" do
+      context "with `destroy!`" do
         subject { Timecop.freeze(destroyed_time) { employee.destroy! } }
 
         before { allow_any_instance_of(Employee).to receive('save!').and_raise(ActiveRecord::RecordNotSaved.new("failed")) }
@@ -2353,7 +2353,7 @@ class EmployeeWithUniquness < Employee
         }
       end
 
-      context "nexted call `.valid_at`" do
+      context "nested call `.valid_at`" do
         before { Employee.create(valid_from: "2019/1/1") }
         it do
           ActiveRecord::Bitemporal.ignore_valid_datetime {
@@ -2368,7 +2368,7 @@ class EmployeeWithUniquness < Employee
         end
       end
 
-      context "nexted call `.valid_at!`" do
+      context "nested call `.valid_at!`" do
         before { Employee.create(valid_from: "2019/1/1") }
         it do
           ActiveRecord::Bitemporal.ignore_valid_datetime {
@@ -2513,7 +2513,7 @@ class EmployeeWithUniquness < Employee
         }
       end
 
-      context "nexted call `.transaction_at`" do
+      context "nested call `.transaction_at`" do
         before { Employee.create(transaction_from: "2019/1/1") }
         it do
           ActiveRecord::Bitemporal.ignore_transaction_datetime {
@@ -2528,7 +2528,7 @@ class EmployeeWithUniquness < Employee
         end
       end
 
-      context "nexted call `.transaction_at!`" do
+      context "nested call `.transaction_at!`" do
         before { Employee.create(transaction_from: "2019/1/1") }
         it do
           ActiveRecord::Bitemporal.ignore_transaction_datetime {
@@ -2720,7 +2720,7 @@ class EmployeeWithUniquness < Employee
         company.employees.third.address = Address.create(city: "Saitama")
       end
 
-      # MEMO: `belongs_to :comany` is alos included.
+      # MEMO: `belongs_to :company` is also included.
       it { expect(company.each_association(deep: true).count).to eq 7 }
       it { expect(company.employees.includes(:address).first.each_association(deep: true).count).to eq 7 }
       it { expect(Company.find(company.id).employees.first.each_association(deep: true).count).to eq 7 }

spec/activerecord-bitemporal/date_type_valid_time_spec.rb

@@ -821,7 +821,7 @@ def self.name
   end
 
   describe "Association" do
-    describe "Date type valid time has many Date type vaild time" do
+    describe "Date type valid time has many Date type valid time" do
       describe "sync valid_from in create" do
         let!(:cop1) { department.cops.new(name: "Jane") }
         let!(:cop2) { department.cops.new(name: "Homu") }
@@ -897,7 +897,7 @@ def self.name
       end
     end
 
-    describe "Date type valid time has many Datetime(timestamp) type vaild time" do
+    describe "Date type valid time has many Datetime(timestamp) type valid time" do
       describe "sync valid_from in create" do
         let!(:manager1) { department.managers.new(name: "Jane") }
         let!(:manager2) { department.managers.new(name: "Homu") }