Improve comments in sync_default_gems
This commit is contained in:
parent
cdbc70d088
commit
544488f114
@ -591,28 +591,28 @@ module SyncDefaultGems
|
|||||||
system(*%w[git add yarp])
|
system(*%w[git add yarp])
|
||||||
end
|
end
|
||||||
|
|
||||||
|
# Skip this commit if everything has been removed as `ignored_paths`.
|
||||||
|
changes = pipe_readlines(%W"git status --porcelain -z")
|
||||||
|
if changes.empty?
|
||||||
|
`git reset` && `git checkout .` && `git clean -fd`
|
||||||
|
puts "Skip empty commit #{sha}"
|
||||||
|
next
|
||||||
|
end
|
||||||
|
|
||||||
|
# For YARP, we want to skip DD: deleted by both.
|
||||||
|
if gem == "yarp"
|
||||||
|
deleted = changes.grep(/^DD /)
|
||||||
|
deleted.map! { |line| line.delete_prefix("DD ") }
|
||||||
|
system(*%W"git rm -f --", *deleted) unless deleted.empty?
|
||||||
|
end
|
||||||
|
|
||||||
# Discover unmerged files
|
# Discover unmerged files
|
||||||
# AU: unmerged, added by us
|
# AU: unmerged, added by us
|
||||||
# DU: unmerged, deleted by us
|
# DU: unmerged, deleted by us
|
||||||
# UU: unmerged, both modified
|
# UU: unmerged, both modified
|
||||||
# UA: unmerged, added by them
|
# UA: unmerged, added by them
|
||||||
# AA: unmerged, both added
|
# AA: unmerged, both added
|
||||||
unmerged = pipe_readlines(%W"git status --porcelain -z")
|
unmerged = changes.map {|line| line[/\A(?:.U|[UA]A) (.*)/, 1]}
|
||||||
if unmerged.empty?
|
|
||||||
# Everything was removed as `ignored_paths`. Skip this commit.
|
|
||||||
`git reset` && `git checkout .` && `git clean -fd`
|
|
||||||
puts "Skip empty commit #{sha}"
|
|
||||||
next
|
|
||||||
end
|
|
||||||
|
|
||||||
# For YARP, we want to handle DD: deleted by both.
|
|
||||||
if gem == "yarp"
|
|
||||||
deleted = unmerged.grep(/^DD /)
|
|
||||||
deleted.map! { |line| line.delete_prefix("DD ") }
|
|
||||||
system(*%W"git rm -f --", *deleted) unless deleted.empty?
|
|
||||||
end
|
|
||||||
|
|
||||||
unmerged.map! {|line| line[/\A(?:.U|[UA]A) (.*)/, 1]}
|
|
||||||
unmerged.compact!
|
unmerged.compact!
|
||||||
ignore, conflict = unmerged.partition {|name| ignore_file_pattern =~ name}
|
ignore, conflict = unmerged.partition {|name| ignore_file_pattern =~ name}
|
||||||
# Reset ignored files if they conflict
|
# Reset ignored files if they conflict
|
||||||
|
Loading…
x
Reference in New Issue
Block a user