diff --git a/changelog_generator.rb b/changelog_generator.rb index 5f0bb2a..44c8c83 100755 --- a/changelog_generator.rb +++ b/changelog_generator.rb @@ -7,15 +7,9 @@ require_relative 'parser' def run_generator options generator = LogGenerator.new(options) - tag_time = generator.find_prev_tag_date - pull_requests = generator.get_all_closed_pull_requests + prev_tag = generator.find_prev_tag - pull_requests.delete_if { |req| - t = Time.parse(req[:closed_at]).utc - t < tag_time - } - - generator.compund_changelog(tag_time, pull_requests) + generator.compund_changelog(prev_tag) end if __FILE__ == $0 diff --git a/log_generator.rb b/log_generator.rb index 9f173e9..07eb480 100644 --- a/log_generator.rb +++ b/log_generator.rb @@ -5,7 +5,7 @@ require 'httparty' class LogGenerator - attr_accessor :options + attr_accessor :options, :all_tags def initialize(options = {}) @options = options @@ -14,6 +14,7 @@ class LogGenerator else @github = Github.new end + @all_tags = self.get_all_tags end def print_json(json) @@ -25,27 +26,9 @@ class LogGenerator %x[#{exec_cmd}] end - def find_prev_tag_date - - value1 = exec_command "git log --tags --simplify-by-decoration --pretty=\"format:%ci %d\" | grep tag" - unless value1 - puts 'not found this tag' - exit - end - - scan_results = value1.scan(/.*tag.*/) - - prev_tag = scan_results[1] - - unless scan_results.any? - puts 'Not found any versions -> exit' - exit - end - - if @options[:verbose] - puts "Prev tag is #{prev_tag}" - end - time = Time.parse(prev_tag) + def find_prev_tag + var = self.all_tags[1] + p var end @@ -65,22 +48,14 @@ class LogGenerator end - def compund_changelog(tag_time, pull_requests) + def compund_changelog(prev_tag) if @options[:verbose] puts 'Generating changelog:' end log = '' - last_tag = exec_command('git describe --abbrev=0 --tags').strip - log += "## [#{last_tag}] (https://github.com/#{$github_user}/#{$github_repo_name}/tree/#{last_tag})\n" - - time_string = tag_time.strftime "%Y/%m/%d" - log += "#### #{time_string}\n" - - pull_requests.each { |dict| - merge = "#{dict[:title]} ([\\##{dict[:number]}](https://github.com/#{$github_user}/#{$github_repo_name}/pull/#{dict[:number]}))\n" - log += "- #{merge}" - } + last_tag = self.all_tags[0] + log += self.generate_log_between_tags(prev_tag, last_tag) puts log File.open('output.txt', 'w') { |file| file.write(log) } @@ -111,8 +86,38 @@ class LogGenerator "User-Agent" => "APPLICATION_NAME"}) json_parse = JSON.parse(response.body) + end - json_parse.each { |obj| p obj['name'] } + def generate_log_between_tags(prev_tag, last_tag) + + last_tag_name = last_tag['name'] + log = '' + prev_tag_time = self.get_time_of_tag(prev_tag) + pull_requests = self.get_all_closed_pull_requests + + pull_requests.delete_if { |req| + t = Time.parse(req[:closed_at]).utc + t < prev_tag_time + } + + log += "## [#{last_tag_name}] (https://github.com/#{$github_user}/#{$github_repo_name}/tree/#{last_tag_name})\n" + + time_string = prev_tag_time.strftime "%Y/%m/%d" + log += "#### #{time_string}\n" + + pull_requests.each { |dict| + merge = "#{dict[:title]} ([\\##{dict[:number]}](https://github.com/#{$github_user}/#{$github_repo_name}/pull/#{dict[:number]}))\n" + log += "- #{merge}" + } + + log + end + + def get_time_of_tag(prev_tag) + github_git_data_commits_get = @github.git_data.commits.get $github_user, $github_repo_name, prev_tag['commit']['sha'] + self.print_json github_git_data_commits_get.body + time_string = github_git_data_commits_get['committer']['date'] + Time.parse(time_string) end @@ -122,7 +127,7 @@ if __FILE__ == $0 log_generator = LogGenerator.new({:verbose => true}) - tags = log_generator.get_all_tags - p tags + tags = log_generator.all_tags + log_generator.generate_log_between_tags(tags[1], tags[2]) end \ No newline at end of file