Workflow support (#183)

Author: cretz

.github/workflows/ci.yml

@@ -9,7 +9,7 @@ on:
 jobs:
   build-lint-test:
     strategy:
-      fail-fast: true
+      fail-fast: false
       matrix:
         # TODO(cretz): Enable Linux ARM. It's not natively supported with setup-ruby (see
         # https://github.com/ruby/setup-ruby#supported-platforms and https://github.com/ruby/setup-ruby/issues/577).
@@ -24,7 +24,7 @@ jobs:
         # https://github.com/temporalio/sdk-ruby/issues/172
         os: [ubuntu-latest, macos-latest]
         # Earliest and latest supported
-        rubyVersion: ["3.1", "3.3"]
+        rubyVersion: ["3.2", "3.3"]
 
         include:
           - os: ubuntu-latest
@@ -73,4 +73,6 @@ jobs:
 
       - name: Lint, compile, test Ruby
         working-directory: ./temporalio
+        # Timeout just in case there's a hanging part in rake
+        timeout-minutes: 20
         run: bundle exec rake TESTOPTS="--verbose"

README.md

@@ -28,7 +28,8 @@ Layout/LeadingCommentSpace:
 # Don't need super for activities
 Lint/MissingSuper:
   AllowedParentClasses:
-    - Temporalio::Activity
+    - Temporalio::Activity::Definition
+    - Temporalio::Workflow::Definition
 
 # Allow tests to nest methods
 Lint/NestedMethodDefinition:
@@ -61,7 +62,7 @@ Metrics/ModuleLength:
 
 # The default is too small
 Metrics/PerceivedComplexity:
-  Max: 25
+  Max: 40
 
 # We want classes to be documented
 Style/Documentation:
@@ -80,6 +81,10 @@ Style/GlobalVars:
   Exclude:
     - test/**/*
 
+# We're ok with two compound comparisons before doing Array.include?
+Style/MultipleComparison:
+  ComparisonsThreshold: 3
+
 # We want our require lists to be in order
 Style/RequireOrder:
   Enabled: true

temporalio/.rubocop.yml

@@ -0,0 +1,2 @@
+--readme README.md
+--protected

temporalio/.yardopts

@@ -17,7 +17,7 @@ group :development do
   gem 'rbs', '~> 3.5.3'
   gem 'rb_sys', '~> 0.9.63'
   gem 'rubocop'
-  gem 'sqlite3', '~> 1.4'
+  gem 'sqlite3'
   gem 'steep', '~> 1.7.1'
   gem 'yard'
 end