ruby-changes:74070
From: Nobuyoshi <ko1@a...>
Date: Tue, 18 Oct 2022 11:39:32 +0900 (JST)
Subject: [ruby-changes:74070] ab3b1b2381 (master): sync_default_gems.rb: fix links to GitHub issues
https://git.ruby-lang.org/ruby.git/commit/?id=ab3b1b2381 From ab3b1b23811c44f56f3dff318b08a9112cba3582 Mon Sep 17 00:00:00 2001 From: Nobuyoshi Nakada <nobu@r...> Date: Tue, 18 Oct 2022 11:30:56 +0900 Subject: sync_default_gems.rb: fix links to GitHub issues - Substitute `GH-xxxx` which does not contain `#` too. - Split each substitutions. --- tool/sync_default_gems.rb | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tool/sync_default_gems.rb b/tool/sync_default_gems.rb index d585f5f14a..a5f8c7fd09 100755 --- a/tool/sync_default_gems.rb +++ b/tool/sync_default_gems.rb @@ -422,13 +422,16 @@ def message_filter(repo, sha) https://github.com/ruby/ruby/blob/trunk/tool/sync_default_gems.rb#L422 log = STDIN.read log.delete!("\r") url = "https://github.com/#{repo}" - print "[#{repo}] ", log.gsub(/\b(?:(?i:fix(?:e[sd])?) +|GH-)\K#(?=\d+\b)|\(\K#(?=\d+\))/) { + log.gsub!(/\b(?:(?i:fix(?:e[sd])?) +)\K#(?=\d+\b)|\bGH-#?(?=\d+\b)|\(\K#(?=\d+\))/) { "#{url}/pull/" - }.gsub(%r{(?<![-\[\](){}\w@/])(?:(\w+(?:-\w+)*/\w+(?:-\w+)*)@)?(\h{10,40})\b}) {|c| + } + log.gsub!(%r{(?<![-\[\](){}\w@/])(?:(\w+(?:-\w+)*/\w+(?:-\w+)*)@)?(\h{10,40})\b}) {|c| "https://github.com/#{$1 || repo}/commit/#{$2[0,12]}" - }.sub(/\s*(?=(?i:\nCo-authored-by:.*)*\Z)/) { + } + log.sub!(/\s*(?=(?i:\nCo-authored-by:.*)*\Z)/) { "\n\n" "#{url}/commit/#{sha[0,10]}\n" } + print "[#{repo}] ", log end # NOTE: This method is also used by ruby-commit-hook/bin/update-default-gem.sh -- cgit v1.2.3 -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/