Skip to content

Commit

Permalink
Merge pull request #144 from Earlopain/drop-base64
Browse files Browse the repository at this point in the history
Drop `base64` dependency, resolve Ruby warning
  • Loading branch information
amatsuda authored Sep 8, 2024
2 parents 7d38ee9 + 11c8c0f commit 45362b4
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 5 deletions.
1 change: 0 additions & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -28,5 +28,4 @@ group :development do
gem "uglifier"
end

gem "base64" if RUBY_VERSION >= "3.3"
gem "logger" if RUBY_VERSION >= "3.4"
2 changes: 0 additions & 2 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ GEM
remote: https://rubygems.org/
specs:
ast (2.4.2)
base64 (0.2.0)
concurrent-ruby (1.3.4)
docile (1.4.1)
execjs (2.9.1)
Expand Down Expand Up @@ -75,7 +74,6 @@ PLATFORMS
universal-java-1.8

DEPENDENCIES
base64
logger
minitest
rake (>= 11)
Expand Down
3 changes: 1 addition & 2 deletions lib/simplecov-html.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
require "fileutils"
require "digest/sha1"
require "time"
require "base64"

# Ensure we are using a compatible version of SimpleCov
major, minor, patch = SimpleCov::VERSION.scan(/\d+/).first(3).map(&:to_i)
Expand Down Expand Up @@ -94,7 +93,7 @@ def assets_path(name)

def asset_inline(name)
path = File.join(@public_assets_dir, name)
base64_content = Base64.strict_encode64 File.read(path)
base64_content = [File.read(path)].pack("m0")

content_type = CONTENT_TYPES[File.extname(name)]

Expand Down

0 comments on commit 45362b4

Please sign in to comment.