admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Merge pull request #3969 from GauthamGoli/release-notes-args

Browse Source 
release-notes: Use Parser to parse args
This commit is contained in:
Mike McQuaid 2018-03-24 16:20:47 +00:00 committed by GitHub
commit 08b2add372
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Library/Homebrew/dev-cmd/release-notes.rb
View File

@ -5,10 +5,16 @@
#: #:
#: If `--markdown` is passed, output as a Markdown list. #: If `--markdown` is passed, output as a Markdown list.
require "cli_parser"
module Homebrew module Homebrew
module_function module_function
def release_notes def release_notes
args = Homebrew::CLI::Parser.new do
switch "--markdown"
end.parse
previous_tag = ARGV.named.first previous_tag = ARGV.named.first
previous_tag ||= Utils.popen_read("git tag --list --sort=-version:refname") previous_tag ||= Utils.popen_read("git tag --list --sort=-version:refname")
.lines.first.chomp .lines.first.chomp
@ -28,7 +34,7 @@ module Homebrew
s.gsub(%r{.*Merge pull request #(\d+) from ([^/]+)/[^>]*(>>)*}, s.gsub(%r{.*Merge pull request #(\d+) from ([^/]+)/[^>]*(>>)*},
"https://github.com/Homebrew/brew/pull/\\1 (@\\2)") "https://github.com/Homebrew/brew/pull/\\1 (@\\2)")
end end
if ARGV.include?("--markdown") if args.markdown?
output.map! do |s| output.map! do |s|
/(.*\d)+ \(@(.+)\) - (.*)/ =~ s /(.*\d)+ \(@(.+)\) - (.*)/ =~ s
"- [#{Regexp.last_match(3)}](#{Regexp.last_match(1)}) (@#{Regexp.last_match(2)})" "- [#{Regexp.last_match(3)}](#{Regexp.last_match(1)}) (@#{Regexp.last_match(2)})"