Skip to content

Commit

Permalink
retire Ruby 2 + gems update + release 1.6 (#91)
Browse files Browse the repository at this point in the history
* Update lockfiles ruby-update

* fix standardrb

Signed-off-by: Thomas Leclaire <[email protected]>

* moved constants at the top of module

* remove unsupported ruby version

* update standardrb

* require ruby version

* new gem version + minimal ruby version

---------

Signed-off-by: Thomas Leclaire <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Thomas Leclaire <[email protected]>
  • Loading branch information
3 people authored Sep 14, 2023
1 parent d8a4b91 commit 1099e2a
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 17 deletions.
10 changes: 1 addition & 9 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,13 +32,6 @@ commands:
destination: test-results

jobs:
build_2_x:
docker:
- image: ruby:2.7.8
working_directory: ~/repo
steps:
- build

build_3_x:
docker:
- image: ruby:3.2.2
Expand All @@ -47,8 +40,7 @@ jobs:
- build

workflows:
version: 2
version: 3
build:
jobs:
- build_2_x
- build_3_x
1 change: 1 addition & 0 deletions .ruby-version
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
3.2.2
2 changes: 1 addition & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ source "https://rubygems.org"
# Specify your gem's dependencies in bing_ads_ruby_sdk.gemspec
gemspec

gem "standard", "1.30.1"
gem "standard", "1.31.1"
2 changes: 1 addition & 1 deletion bing_ads_ruby_sdk.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ require "bing_ads_ruby_sdk/version"

Gem::Specification.new do |spec|
spec.name = "bing_ads_ruby_sdk"
spec.required_ruby_version = ">= 2.3"
spec.required_ruby_version = ">= 3"

spec.version = BingAdsRubySdk::VERSION
spec.authors = %w[Effilab developers]
Expand Down
10 changes: 5 additions & 5 deletions lib/bing_ads_ruby_sdk.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,9 @@
require "bing_ads_ruby_sdk/railtie" if defined?(Rails)

module BingAdsRubySdk
TYPE_KEY = "@type"
ROOT_PATH = File.join(__dir__, "..")

def self.config
@configuration ||= BingAdsRubySdk::Configuration.new
end
Expand All @@ -18,9 +21,9 @@ def self.configure
yield(config)
end

def self.log(level, *args, &block)
def self.log(level, *, &block)
return unless config.log
config.logger.send(level, *args, &block)
config.logger.send(level, *, &block)
end

def self.root_path
Expand All @@ -30,7 +33,4 @@ def self.root_path
def self.type_key
TYPE_KEY
end

TYPE_KEY = "@type"
ROOT_PATH = File.join(__dir__, "..")
end
2 changes: 1 addition & 1 deletion lib/bing_ads_ruby_sdk/version.rb
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# frozen_string_literal: true

module BingAdsRubySdk
VERSION = "1.5.2"
VERSION = "1.6"
DEFAULT_SDK_VERSION = :v13
end

0 comments on commit 1099e2a

Please sign in to comment.