Require appsignal in Grape integration file (#1122)

Make people's lives easier by requiring appsignal from the Grape integration file so they don't have to require two files. Also add a log message to help debug if the Grape integration file is loaded like we do for other integrations. [skip review]
appsignal · Jun 27, 2024 · b3a8003 · b3a8003
1 parent 66bb7a6
commit b3a8003
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/.changesets/require-appsignal-in-grape-integration-file.md b/.changesets/require-appsignal-in-grape-integration-file.md
@@ -0,0 +1,6 @@
+---
+bump: patch
+type: change
+---
+
+Require the AppSignal gem in the Grape integration file. Previously `require "appsignal"` had to be called before `require "appsignal/integrations/grape"`. This `require "appsignal"` is no longer required.
diff --git a/lib/appsignal/integrations/grape.rb b/lib/appsignal/integrations/grape.rb
@@ -1,7 +1,10 @@
 # frozen_string_literal: true
 
+require "appsignal"
 require "appsignal/rack/grape_middleware"
 
+Appsignal.internal_logger.debug("Loading Grape integration")
+
 module Appsignal
   # @api private
   module Grape