On Fri, Aug 21, 2020 at 01:40:52PM +0800, Bai Jing wrote:
On Fri, Aug 21, 2020 at 01:20:07PM +0800, Wu Fengguang wrote:
In title, "todo: " is too general. It's already todo repo.
Please mention the error it fixed.
Ok, I will add changelog.
- %x(git -C ~/.am/#{repo} pull --rebase) - %x(cd ~/.am/#{repo} && git shortlog -ens --no-merges --since "#{days} days ago" -- #{files}).each_line do |line|
No need change the above 2 lines?
Ok, I got it.
+ %x(cd #{ENV['HOME']}/.am/ && git clone file:///c/#{repo}.git) unless File.exist?("#{ENV['HOME']}/.am/#{repo}") + %x(git -C #{ENV['HOME']}/.am/#{repo} pull --rebase 2>&1) + file = %x(cd #{ENV['HOME']}/.am/#{repo} && git shortlog -ens --no-merges --since "#{days} days ago" -- #{files}) + file.each_line do |line|
I used rubocop to check this script. It shows this line is too long. So I used a variable. "%x(cd ~/.am/#{repo} && git shortlog -ens --no-merges --since "#{days} days ago" -- #{files}).each_line do |line|" Thanks, Baijing
if line =~ /(\d+)\t(.*) <(.*)>/ count = $1.to_i name = $2 -- 2.23.0