Merge pull request #265 from joshbuker/update/version-requirements

Update Version Requirements to resolve CircleCI failures

Author: swarley

.circleci/config.yml

@@ -6,7 +6,7 @@ commands:
     steps:
       - run:
           name: Install OS packages
-          command: apk add git build-base ruby-dev ruby-etc ruby-json libsodium
+          command: apk add git build-base ruby-dev ruby-etc ruby-json libsodium-dev
       - checkout
       - run:
           name: "Ruby version"
@@ -27,9 +27,9 @@ commands:
             - ./vendor/bundle
 
 jobs:
-  test_ruby_27:
+  test_ruby_31:
     docker:
-      - image: ruby:2.7-alpine
+      - image: ruby:3.1-alpine
     steps:
       - setup-env
       - run:
@@ -38,9 +38,9 @@ jobs:
       - store_test_results:
           path: ~/rspec
 
-  test_ruby_30:
+  test_ruby_32:
     docker:
-      - image: ruby:3.0-alpine
+      - image: ruby:3.2-alpine
     steps:
       - setup-env
       - run:
@@ -50,9 +50,9 @@ jobs:
           path: ~/rspec
 
 
-  test_ruby_31:
+  test_ruby_33:
     docker:
-      - image: ruby:3.1-alpine
+      - image: ruby:3.3-alpine
     steps:
       - setup-env
       - run:
@@ -61,7 +61,7 @@ jobs:
 
   rubocop:
     docker:
-      - image: ruby:2.7-alpine
+      - image: ruby:3.3-alpine
     steps:
       - setup-env
       - run:
@@ -70,7 +70,7 @@ jobs:
 
   yard:
     docker:
-      - image: ruby:2.7-alpine
+      - image: ruby:3.3-alpine
     steps:
       - setup-env
       - attach_workspace:
@@ -125,9 +125,9 @@ jobs:
 workflows:
   test:
     jobs:
-      - test_ruby_27
-      - test_ruby_30
       - test_ruby_31
+      - test_ruby_32
+      - test_ruby_33
       - rubocop
       - yard
   deploy:

.devcontainer/Dockerfile

@@ -0,0 +1,13 @@
+# Which Ruby version to use. You may need to use a more restrictive version,
+# e.g. `3.0`
+ARG VARIANT=3.3
+
+# Pull Microsoft's ruby devcontainer base image
+FROM mcr.microsoft.com/devcontainers/ruby:${VARIANT}
+
+# Install libsodium dependency for voice channel interactions
+RUN apt update -yq && apt install -y libsodium-dev
+
+# Ensure we're running the latest bundler, as what ships with the Ruby image may
+# not be current, and bundler will auto-downgrade to match the Gemfile.lock
+RUN gem install bundler

.devcontainer/devcontainer.json

@@ -0,0 +1,29 @@
+{
+  "name": "Ruby",
+  "build": {
+    "dockerfile": "Dockerfile"
+  },
+
+  // Configure tool-specific properties.
+  "customizations": {
+    // Configure properties specific to VS Code.
+    "vscode": {
+      // Add the IDs of extensions you want installed when the container is created.
+      "extensions": [
+        "shopify.ruby-lsp"
+      ]
+    }
+  },
+
+  // Set the environment variables
+  // "runArgs": ["--env-file",".env"],
+
+  // Use 'forwardPorts' to make a list of ports inside the container available locally.
+  //   "forwardPorts": [],
+
+  // Use 'postCreateCommand' to run commands after the container is created.
+  "postCreateCommand": "bash .devcontainer/postcreate.sh",
+
+  // Set `remoteUser` to `root` to connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
+  "remoteUser": "vscode"
+}

.devcontainer/postcreate.sh

@@ -0,0 +1,4 @@
+#!/bin/bash
+
+bundle config set path vendor/bundle
+bundle install --jobs=1

.rubocop.yml

@@ -8,7 +8,7 @@ inherit_mode:
 
 AllCops:
   NewCops: enable
-  TargetRubyVersion: 2.7
+  TargetRubyVersion: 3.3
 
 # Disable line length checks
 Layout/LineLength:
@@ -66,3 +66,53 @@ Style/SingleLineBlockParams:
   Methods:
     - reduce: [m, e]
     - inject: [m, e]
+
+###################################
+## NEW COPS TO MAKE DECISIONS ON ##
+###################################
+
+# TODO: Decide how you want to handle this:
+# https://rubydoc.info/gems/rubocop/RuboCop/Cop/Style/HashSyntax
+Style/HashSyntax:
+  EnforcedShorthandSyntax: either
+
+Style/ArgumentsForwarding:
+  Enabled: false
+
+Performance/StringIdentifierArgument:
+  Enabled: false
+
+Style/ComparableClamp:
+  Enabled: false
+
+Style/SuperArguments:
+  Enabled: false
+
+Naming/BlockForwarding:
+  Enabled: false
+
+Style/RedundantSelfAssignmentBranch:
+  Enabled: false
+
+Performance/MapCompact:
+  Enabled: false
+
+Gemspec/RequiredRubyVersion:
+  Enabled: false
+
+Gemspec/DevelopmentDependencies:
+  Enabled: false
+
+Style/RedundantReturn:
+  Enabled: false
+
+Style/RedundantParentheses:
+  Enabled: false
+
+# This will probably be a breaking change, but should happen
+Style/ReturnNilInPredicateMethodDefinition:
+  Enabled: false
+
+# FIXME: Disabled due to breaking tests, should probably refactor the code instead
+Style/SafeNavigation:
+  Enabled: false

README.md

@@ -44,7 +44,7 @@ If you enjoy using the library, consider getting involved with the community to
 
 ## Dependencies
 
-* Ruby >= 2.7 supported
+* Ruby >= 3.1 supported
 * An installed build system for native extensions (on Windows, make sure you download the "Ruby+Devkit" version of [RubyInstaller](https://rubyinstaller.org/downloads/))
 
 ### Voice dependencies

discordrb-webhooks.gemspec

@@ -22,7 +22,7 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency 'rest-client', '>= 2.0.0'
 
-  spec.required_ruby_version = '>= 2.7'
+  spec.required_ruby_version = '>= 3.1'
   spec.metadata = {
     'rubygems_mfa_required' => 'true'
   }

discordrb.gemspec

@@ -19,7 +19,10 @@ Gem::Specification.new do |spec|
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.metadata = {
+    'bug_tracker_uri' => 'https://github.com/shardlab/discordrb/issues',
     'changelog_uri' => 'https://github.com/shardlab/discordrb/blob/main/CHANGELOG.md',
+    'documentation_uri' => 'https://github.com/shardlab/discordrb/wiki',
+    'source_code_uri' => 'https://github.com/shardlab/discordrb',
     'rubygems_mfa_required' => 'true'
   }
   spec.require_paths = ['lib']
@@ -31,17 +34,17 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency 'discordrb-webhooks', '~> 3.5.0'
 
-  spec.required_ruby_version = '>= 2.7'
+  spec.required_ruby_version = '>= 3.1'
 
   spec.add_development_dependency 'bundler', '>= 1.10', '< 3'
   spec.add_development_dependency 'rake', '~> 13.0'
-  spec.add_development_dependency 'redcarpet', '~> 3.5.0' # YARD markdown formatting
-  spec.add_development_dependency 'rspec', '~> 3.11.0'
-  spec.add_development_dependency 'rspec_junit_formatter', '~> 0.5.1'
-  spec.add_development_dependency 'rspec-prof', '~> 0.0.7'
-  spec.add_development_dependency 'rubocop', '~> 1.36.0'
+  spec.add_development_dependency 'redcarpet', '~> 3.5' # YARD markdown formatting
+  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'rspec_junit_formatter', '~> 0.5'
+  spec.add_development_dependency 'rspec-prof', '~> 0.0'
+  spec.add_development_dependency 'rubocop', '~> 1.0'
   spec.add_development_dependency 'rubocop-performance', '~> 1.0'
-  spec.add_development_dependency 'rubocop-rake', '~> 0.6.0'
-  spec.add_development_dependency 'simplecov', '~> 0.21.0'
-  spec.add_development_dependency 'yard', '~> 0.9.9'
+  spec.add_development_dependency 'rubocop-rake', '~> 0.6'
+  spec.add_development_dependency 'simplecov', '~> 0.21'
+  spec.add_development_dependency 'yard', '~> 0.9'
 end

lib/discordrb/bot.rb

@@ -2,7 +2,6 @@
 
 require 'rest-client'
 require 'zlib'
-require 'set'
 
 require 'discordrb/events/message'
 require 'discordrb/events/typing'
@@ -1700,7 +1699,7 @@ def call_event(handler, event)
 
     def handle_awaits(event)
       @awaits ||= {}
-      @awaits.each do |_, await|
+      @awaits.each_value do |await|
         key, should_delete = await.match(event)
         next unless key
 

lib/discordrb/commands/parser.rb

@@ -159,7 +159,7 @@ def execute_bare(event)
       escaped = false
       hacky_delim, hacky_space, hacky_prev, hacky_newline = [0xe001, 0xe002, 0xe003, 0xe004].pack('U*').chars
 
-      @chain.each_char.each_with_index do |char, index|
+      @chain.each_char.with_index do |char, index|
         # Escape character
         if char == '\\' && !escaped
           escaped = true