Skip to content

Commit dea1788

Browse files
zkoppertCopilot
andcommitted
build: drop unused sanitize gem dev dependency
html-pipeline v3 uses Selma directly instead of the sanitize gem, and nothing in this repo requires sanitize. The 'sanitize' method calls in lib/github/markup/command_implementation.rb are an internal encoding helper, not the gem. Caught by Copilot's code review on #2061. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: Zack Koppert <zkoppert@github.com>
1 parent e7beefb commit dea1788

2 files changed

Lines changed: 0 additions & 6 deletions

File tree

Gemfile.lock

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ GEM
3535
commonmarker (2.8.2-x86_64-linux)
3636
concurrent-ruby (1.3.6)
3737
connection_pool (3.0.2)
38-
crass (1.0.6)
3938
creole (0.5.0)
4039
date (3.5.1)
4140
docile (1.4.1)
@@ -95,9 +94,6 @@ GEM
9594
rexml (3.4.4)
9695
rubypants (0.7.1)
9796
rugged (1.9.0)
98-
sanitize (6.1.3)
99-
crass (~> 1.0.2)
100-
nokogiri (>= 1.12.0)
10197
securerandom (0.4.1)
10298
selma (0.5.0)
10399
rb_sys (~> 0.9)
@@ -157,7 +153,6 @@ DEPENDENCIES
157153
rdoc (~> 7.2.0)
158154
redcarpet
159155
rexml
160-
sanitize (>= 4.6.3)
161156
simplecov (~> 0.22)
162157
twitter-text (~> 1.14)
163158
wikicloth (= 0.8.3)

github-markup.gemspec

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ Gem::Specification.new do |s|
2727
s.add_development_dependency 'activesupport', '~> 8.1.3'
2828
s.add_development_dependency 'minitest', '>= 5.4.3', '~> 6.0'
2929
s.add_development_dependency 'html-pipeline', '~> 3.2'
30-
s.add_development_dependency 'sanitize', '>= 4.6.3'
3130
s.add_development_dependency 'nokogiri', '~> 1.19.2'
3231
s.add_development_dependency 'nokogiri-diff', '~> 0.3.0'
3332
s.add_development_dependency "github-linguist", ">= 7.1.3"

0 commit comments

Comments
 (0)