will/github-changelog-generator
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add warrning in case of 0 tags in repo. Fix #208. Reformat + prettify output.

Browse Source
This commit is contained in:
Petr Korolev 2015-03-24 13:43:16 +02:00
parent a1c7ffb765
commit 5f6756e8dd
1 changed files with 17 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
lib/github_changelog_generator.rb
View File

@ -16,7 +16,7 @@ module GitHubChangelogGenerator
PER_PAGE_NUMBER = 30 PER_PAGE_NUMBER = 30
GH_RATE_LIMIT_EXCEEDED_MSG = 'Warning: GitHub API rate limit exceed (5000 per hour), change log may not ' + GH_RATE_LIMIT_EXCEEDED_MSG = 'Warning: GitHub API rate limit exceed (5000 per hour), change log may not ' +
'contain some issues. You can limit the number of issues fetched using the `--max-issues NUM` argument' 'contain some issues. You can limit the number of issues fetched using the `--max-issues NUM` argument'
def initialize def initialize
@ -101,7 +101,7 @@ module GitHubChangelogGenerator
end end
issue[:actual_date] = commit[:author][:date] issue[:actual_date] = commit[:author][:date]
rescue rescue
puts "Warning: can't fetch commit #{event[:commit_id]} probably it referenced from another repo." puts "Warning: can't fetch commit #{event[:commit_id]} probably it referenced from another repo.".yellow
issue[:actual_date] = issue[:closed_at] issue[:actual_date] = issue[:closed_at]
end end
end end
@ -153,7 +153,7 @@ module GitHubChangelogGenerator
self.fetch_merged_at_pull_requests self.fetch_merged_at_pull_requests
filtered_pull_requests = @pull_requests.select {|pr| pr[:merged_at] != nil } filtered_pull_requests = @pull_requests.select { |pr| pr[:merged_at] != nil }
unless @options[:include_labels].nil? unless @options[:include_labels].nil?
filtered_pull_requests = @pull_requests.select { |issue| filtered_pull_requests = @pull_requests.select { |issue|
@ -285,7 +285,7 @@ module GitHubChangelogGenerator
threads.each { |thr| thr.join } threads.each { |thr| thr.join }
if @options[:verbose] if @options[:verbose]
puts 'Fetching tags: Done!' puts "Fetching tags dates: #{i} Done!"
end end
end end
@ -316,6 +316,11 @@ module GitHubChangelogGenerator
if @options[:verbose] if @options[:verbose]
puts "Found #{tags.count} tags" puts "Found #{tags.count} tags"
end end
if tags.count == 0
puts "Warning: Can't find any tags in repo. Make sure, that you push tags to remote repo via 'git push --tags'".yellow
end
rescue rescue
puts GH_RATE_LIMIT_EXCEEDED_MSG.yellow puts GH_RATE_LIMIT_EXCEEDED_MSG.yellow
end end
@ -623,12 +628,7 @@ module GitHubChangelogGenerator
fetch_events_async(@issues + @pull_requests) fetch_events_async(@issues + @pull_requests)
#to clear line from prev print
print " \r"
if @options[:verbose]
puts 'Fetching events for issues and PR: Done!'
end
end end
def fetch_events_async(issues) def fetch_events_async(issues)
@ -651,6 +651,14 @@ module GitHubChangelogGenerator
threads.each { |thr| thr.join } threads.each { |thr| thr.join }
threads = [] threads = []
} }
#to clear line from prev print
print " \r"
if @options[:verbose]
puts "Fetching events for issues and PR: #{i} Done!"
end
end end
end end