Merge in changes from the main repo and update for Rails 8 compatibilty

.github/workflows/standardrb.yaml

@@ -0,0 +1,13 @@
+name: StandardRB
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - name: StandardRB Linter
+      uses: standardrb/standard-ruby-action@v0.0.5
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.ruby-version

@@ -1 +1 @@
-2.7.5
+3.2.2

Gemfile

@@ -1,10 +1,11 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in pager_duty-connection.gemspec
 gemspec
 
 # for tests & running examples
 group :development do
-  gem 'dotenv'
-  gem 'pry'
+  gem "dotenv"
+  gem "pry"
+  gem "standardrb"
 end

README.md

@@ -31,7 +31,9 @@ And this is what it doesn't do:
 
 Add this line to your application's Gemfile:
 
-    gem 'pager_duty-connection'
+```ruby
+gem "pager_duty-connection"
+```
 
 And then execute:
 
@@ -54,22 +56,22 @@ pagerduty = PagerDuty::Connection.new(token)
 pagerduty = PagerDuty::Connection.new(token, token_type: :Bearer)
 
 # setup to use a custom domain
-pagerduty = PagerDuty::Connection.new(token, token_type: :Bearer, url: 'https://custom.domain.com')
+pagerduty = PagerDuty::Connection.new(token, token_type: :Bearer, url: "https://custom.domain.com")
 
 # 4 main methods: `get`, `post`, `put`, and `delete`:
 
-response = pagerduty.get('some/relative/path', params)
-response = pagerduty.post('some/relative/path', params)
-response = pagerduty.delete('some/relative/path', params)
-response = pagerduty.put('some/relative/path', params)
+response = pagerduty.get("some/relative/path", params)
+response = pagerduty.post("some/relative/path", params)
+response = pagerduty.delete("some/relative/path", params)
+response = pagerduty.put("some/relative/path", params)
 
 # use something like irb or pry to poke around the responses
 # the contents will vary a bit between call, ie:
 
-response = pagerduty.get('incidents')
+response = pagerduty.get("incidents")
 response.incidents # an array of incidents
 
-response = pagerduty.get('incidents/YYZ')
+response = pagerduty.get("incidents/YYZ")
 response # the hash/object that represents the array
 ```
 
@@ -102,14 +104,14 @@ In general, you can get/put/post/delete a path, with some attributes. Use the [R
 If you are working in Rails, and using only a single PagerDuty account, you'll probably want an initializer:
 
 ```ruby
-$pagerduty = PagerDuty::Connection.new('your-token')
+$pagerduty = PagerDuty::Connection.new("your-token")
 ```
 
 And if you are using [dotenv](https://github.com/bkeepers/dotenv), you can use environment variables, and stash them in .env:
 
 ```ruby
-account = ENV['PAGERDUTY_ACCOUNT'] || raise("Missing ENV['PAGERDUTY_ACCOUNT'], add to .env")
-token = ENV['PAGERDUTY_TOKEN'] || raise("Missing ENV['PAGERDUTY_TOKEN'], add to .env.#{Rails.env}")
+account = ENV["PAGERDUTY_ACCOUNT"] || raise("Missing ENV['PAGERDUTY_ACCOUNT'], add to .env")
+token = ENV["PAGERDUTY_TOKEN"] || raise("Missing ENV['PAGERDUTY_TOKEN'], add to .env.#{Rails.env}")
 $pagerduty = PagerDuty::Connection.new(account, token)
 ```
 

examples/find-users.rb

@@ -1,15 +1,15 @@
 #!/usr/bin/env ruby
 
-require 'dotenv'
-Dotenv.load ".env.development", '.env'
+require "dotenv"
+Dotenv.load ".env.development", ".env"
 
-token = ENV['PAGERDUTY_TOKEN'] || raise("Missing ENV['PAGERDUTY_TOKEN'], add to .env.development")
+token = ENV["PAGERDUTY_TOKEN"] || raise("Missing ENV['PAGERDUTY_TOKEN'], add to .env.development")
 
-require 'pager_duty/connection'
-$pagerduty = PagerDuty::Connection.new(token)
+require "pager_duty/connection"
+pagerduty = PagerDuty::Connection.new(token)
 
 # https://v2.developer.pagerduty.com/v2/page/api-reference#!/Users/get_users
-response = $pagerduty.get('users')
-response['users'].each do |user|
-  puts "#{user['name']}: #{user['email']}"
+response = pagerduty.get("users")
+response["users"].each do |user|
+  puts "#{user["name"]}: #{user["email"]}"
 end

examples/shifts-with-incidents-and-log-entries.rb

@@ -1,46 +1,46 @@
 #!/usr/bin/env ruby
 
-require 'dotenv'
-Dotenv.load ".env.development", '.env'
+require "dotenv"
+Dotenv.load ".env.development", ".env"
 
-token = ENV['PAGERDUTY_TOKEN'] || raise("Missing ENV['PAGERDUTY_TOKEN'], add to .env.development")
+token = ENV["PAGERDUTY_TOKEN"] || raise("Missing ENV['PAGERDUTY_TOKEN'], add to .env.development")
 
-require 'pager_duty/connection'
-$pagerduty = PagerDuty::Connection.new(token)
+require "pager_duty/connection"
+pagerduty = PagerDuty::Connection.new(token)
 
-schedule_id = ENV['PAGERDUTY_SCHEDULE_ID'] || raise("Missing ENV['PAGERDUTY_SCHEDULE_ID'], add to .env.development")
+schedule_id = ENV["PAGERDUTY_SCHEDULE_ID"] || raise("Missing ENV['PAGERDUTY_SCHEDULE_ID'], add to .env.development")
 
 # pull down schedule entires for XXX schedule in the last day (ie who has been on call, and when
 time_since = 1.day.ago
 time_until = Time.now
 
 # https://v2.developer.pagerduty.com/v2/page/api-reference#!/On-Calls/get_oncalls
-response = $pagerduty.get("oncalls", query_params: { since: time_since, until: time_until, schedule_ids: [schedule_id] })
+response = pagerduty.get("oncalls", query_params: {since: time_since, until: time_until, schedule_ids: [schedule_id]})
 
-entries = response['oncalls']
+entries = response["oncalls"]
 
 entries.each do |entry|
-  puts "#{entry['start']} - #{entry['end']}: #{entry['user']['summary']}"
+  puts "#{entry["start"]} - #{entry["end"]}: #{entry["user"]["summary"]}"
 
   # find incidents during that shift
   # https://v2.developer.pagerduty.com/v2/page/api-reference#!/Incidents/get_incidents
-  response = $pagerduty.get('incidents', query_params: { since: entry['start'], until: entry['end'], user_ids: [entry['user']['id']] })
+  response = pagerduty.get("incidents", query_params: {since: entry["start"], until: entry["end"], user_ids: [entry["user"]["id"]]})
 
-  response['incidents'].each do |incident|
+  response["incidents"].each do |incident|
     puts "\t#{incident.id}"
 
     # find log entries (acknowledged, notifications, etc) for incident:
     # https://v2.developer.pagerduty.com/v2/page/api-reference#!/Incidents/get_incidents_id_log_entries
-    response = $pagerduty.get("incidents/#{incident.id}/log_entries")
+    response = pagerduty.get("incidents/#{incident.id}/log_entries")
 
     # select just the notes
-    notes = response['log_entries'].select do |log_entry|
-      log_entry['channel'] && log_entry['channel']['type'] == 'note'
+    notes = response["log_entries"].select do |log_entry|
+      log_entry["channel"] && log_entry["channel"]["type"] == "note"
     end
 
     # and print them out:
     notes.each do |log_entry|
-      puts "\t\t#{log_entry['channel']['summary']}"
+      puts "\t\t#{log_entry["channel"]["summary"]}"
     end
   end
 end

lib/pager_duty.rb

@@ -1,6 +1,5 @@
-require 'pager_duty/connection/version'
+require "pager_duty/connection/version"
 
 module PagerDuty
-  autoload :Connection, 'pager_duty/connection'
+  autoload :Connection, "pager_duty/connection"
 end
-