ownership 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 24115b22ad0814c98dbb02a7fe2fb84af7149d9b8d2e4a052b05bba5b650cf54
-  data.tar.gz: 48b57d21ec9275b84aed0c4250a29ec1833688c737b2d5b2ec77c2969ea245cb
+  metadata.gz: 0723f75e9bb391d89f88e06c4686efc39b9c6e052cab1c8787412cb084a77bb7
+  data.tar.gz: 2b495a405e852a695ffcc56ba30ce3910ebe4e039afb62aaa9872ca912c5f332
 SHA512:
-  metadata.gz: 7b9efc666b672de029ecc86f083207ef11af64ba62d7c6bc760b1228d8fb9ddc951281d20e91f328073d19306f907e0ac5b0c734b09a6b8a18f96a84dbf715f0
-  data.tar.gz: 8469963791b7750afc38ecc662a5e0af1c4385836bd6af8deb9e6f0d6a31ecc2e0d1f00f01559b1dfcc2a8bf86ab39b6a930d2eb1f3db13bdae3000ebbc71b67
+  metadata.gz: 67fc999361bffcb73ead84fa6fd39fe959a5069309f78060fdf81f2e36cb3b3ef7c1ca04e5a2731278b076eada4860a9c0f23ddc62caef8f8f91413d1978a6ed
+  data.tar.gz: af4ab99285ec7ff5b127dcb715eacade86fd7b78f3d4aae39c65da1cbacccfab1946b4584a4f3c796d13734f5524fd8c5c40048af9b88ef79414002f2197f9f1

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.5.0 (2025-10-26)
+
+- Dropped support for Ruby < 3.2 and Rails < 7.2
+- Dropped support for Marginalia
+
 ## 0.4.0 (2024-10-01)
 
 - Added support for Rails 8

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2017-2023 Andrew Kane
+Copyright (c) 2017-2025 Andrew Kane
 
 MIT License
 

data/README.md

@@ -68,15 +68,15 @@ Ownership.default_owner = :logistics
 There are a few built-in integrations with other gems.
 
 - [Active Record](#active-record)
+- [AppSignal](#appsignal)
 - [Honeybadger](#honeybadger)
-- [Marginalia](#marginalia)
 - [Rollbar](#rollbar)
 
 You can also add [custom integrations](#custom-integrations).
 
 ### Active Record
 
-Active Record 7+ has the option to add comments to queries.
+Active Record has the option to add comments to queries.
 
 ```sql
 SELECT ...
@@ -90,6 +90,12 @@ config.active_record.query_log_tags_enabled = true
 config.active_record.query_log_tags << :owner
 ```
 
+### AppSignal
+
+The [AppSignal gem integrates with Ownership](https://docs.appsignal.com/ruby/integrations/ownership.html) automatically. Error and performance samples in AppSignal will be tagged with the specified owner.
+
+You can set AppSignal's [`ownership_set_namespace` configuration option](https://docs.appsignal.com/ruby/configuration/options.html#option-ownership_set_namespace) to `true` in order to use the specified owner as an AppSignal namespace, which allows you to easily list performance actions and error incidents for each namespace.
+
 ### Honeybadger
 
 [Honeybadger](https://github.com/honeybadger-io/honeybadger-ruby) tracks exceptions. This integration makes it easy to send exceptions to different projects based on the owner. We recommend having a project for each team.
@@ -107,17 +113,6 @@ Also works with a proc
 Ownership::Honeybadger.api_keys = ->(owner) { ENV["#{owner.to_s.upcase}_HONEYBADGER_API_KEY"] }
 ```
 
-### Marginalia
-
-[Marginalia](https://github.com/basecamp/marginalia) adds comments to Active Record queries. If installed, the owner is added.
-
-```sql
-SELECT ...
-/*application:MyApp,controller:posts,action:index,owner:logistics*/
-```
-
-This can be useful when looking at the most time-consuming queries on your database.
-
 ### Rollbar
 
 [Rollbar](https://github.com/rollbar/rollbar-gem) tracks exceptions. This integration makes it easy to send exceptions to different projects based on the owner. We recommend having a project for each team.
@@ -153,8 +148,6 @@ Ownership.around_change = proc do |owner, block|
 end
 ```
 
-Please don’t hesitate to [submit a pull request](https://github.com/ankane/ownership/pulls) if you create an integration that others can use.
-
 Exceptions that bubble up from an `owner` block have the owner, which your exception reporting library can use.
 
 ```ruby
@@ -177,7 +170,7 @@ Thanks to [Nick Elser](https://github.com/nickelser) for creating this pattern.
 
 ## History
 
-View the [changelog](https://github.com/ankane/ownership/blob/master/CHANGELOG.md).
+View the [changelog](https://github.com/ankane/ownership/blob/master/CHANGELOG.md)
 
 ## Contributing
 

data/lib/ownership/controller_methods.rb

@@ -5,8 +5,8 @@ module Ownership
     extend ActiveSupport::Concern
 
     class_methods do
-      def owner(owner, options = {})
-        around_action options do |_, block|
+      def owner(owner, **options)
+        around_action(**options) do |_, block|
           owner(owner) { block.call }
         end
       end

data/lib/ownership/global_methods.rb

@@ -2,12 +2,7 @@ module Ownership
   module GlobalMethods
     private
 
-    def owner(*args, &block)
-      return super if is_a?(Method) # hack for pry
-
-      owner = args[0]
-      # same error message as Ruby
-      raise ArgumentError, "wrong number of arguments (given #{args.size}, expected 1)" if args.size != 1
+    def owner(owner, &block)
       raise ArgumentError, "Missing block" unless block_given?
 
       previous_value = Thread.current[:ownership_owner]

data/lib/ownership/job_methods.rb

@@ -5,9 +5,9 @@ module Ownership
     extend ActiveSupport::Concern
 
     class_methods do
-      def owner(*args)
+      def owner(owner)
         around_perform do |_, block|
-          owner(*args) { block.call }
+          owner(owner) { block.call }
         end
       end
     end

data/lib/ownership/version.rb

@@ -1,3 +1,3 @@
 module Ownership
-  VERSION = "0.4.0"
+  VERSION = "0.5.0"
 end

data/lib/ownership.rb

@@ -18,29 +18,23 @@ end
 
 Object.include Ownership::GlobalMethods
 
-if defined?(ActiveSupport)
+if defined?(ActiveSupport.on_load)
   ActiveSupport.on_load(:action_controller) do
     require_relative "ownership/controller_methods"
     include Ownership::ControllerMethods
   end
 
   ActiveSupport.on_load(:active_record) do
-    if ActiveRecord::VERSION::MAJOR >= 7
-      # taggings is frozen in Active Record 8
-      if !ActiveRecord::QueryLogs.taggings[:owner]
-        ActiveRecord::QueryLogs.taggings = ActiveRecord::QueryLogs.taggings.merge({owner: -> { Ownership.owner }})
-      end
+    # taggings is frozen in Active Record 8
+    if !ActiveRecord::QueryLogs.taggings[:owner]
+      ActiveRecord::QueryLogs.taggings = ActiveRecord::QueryLogs.taggings.merge({owner: -> { Ownership.owner }})
     end
-
-    require_relative "ownership/marginalia" if defined?(Marginalia)
   end
 
   ActiveSupport.on_load(:active_job) do
     require_relative "ownership/job_methods"
     include Ownership::JobMethods
   end
-else
-  require_relative "ownership/marginalia" if defined?(Marginalia)
 end
 
 class Exception

metadata

@@ -1,16 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ownership
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Andrew Kane
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-10-01 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies: []
-description:
 email: andrew@ankane.org
 executables: []
 extensions: []
@@ -24,14 +22,12 @@ files:
 - lib/ownership/global_methods.rb
 - lib/ownership/honeybadger.rb
 - lib/ownership/job_methods.rb
-- lib/ownership/marginalia.rb
 - lib/ownership/rollbar.rb
 - lib/ownership/version.rb
 homepage: https://github.com/ankane/ownership
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -39,15 +35,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.1'
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.16
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Code ownership for Rails
 test_files: []

data/lib/ownership/marginalia.rb

@@ -1,9 +0,0 @@
-module Marginalia
-  module Comment
-    def self.owner
-      Ownership.owner
-    end
-  end
-end
-
-Marginalia::Comment.components << :owner