Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Commit

Permalink
Merge #7190
Browse files Browse the repository at this point in the history
7190: Remove incorrect `man:clobber` task r=deivid-rodriguez a=deivid-rodriguez

### What was the end-user problem that led to this PR?

The problem was that `man:clobber` was listed under `bin/rake -T` but it wasn't working because it was trying to remove a non existent folder.

### What was your diagnosis of the problem?

My diagnosis was that the correct task is actually `bin/rake man:clean`.

### What is your fix for the problem, implemented in this PR?

My fix is to remove `man:clobber` and move its description on top of `man:clean`, so that it's correctly listed under `bin/rake -T`.

Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
  • Loading branch information
bundlerbot and deivid-rodriguez committed Jun 8, 2019
2 parents 5e989c1 + f726426 commit ac832fe
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions Rakefile
Expand Up @@ -242,6 +242,7 @@ namespace :man do
end
task :build_all_pages => "index.txt"

desc "Remove all built man pages"
task :clean do
leftovers = Dir["man/*"].reject do |f|
File.extname(f) == ".ronn"
Expand All @@ -251,11 +252,6 @@ namespace :man do

desc "Build the man pages"
task :build => ["man:clean", "man:build_all_pages"]

desc "Remove all built man pages"
task :clobber do
rm_rf "lib/bundler/man"
end
end
end
end
Expand Down

0 comments on commit ac832fe

Please sign in to comment.