Make a constant for the regex to find hidden files
A raw regular expression isn't very expressive, IMHO. Rather than having people who read this code parse the regular expression to figure out what it's for, let's give a name. This way, it becomes more obvious what exactly it is we're doing here.
This commit is contained in:
parent
8add5aec4b
commit
b72556fd03
|
@ -3,6 +3,7 @@ require 'set'
|
||||||
module Jekyll
|
module Jekyll
|
||||||
# Handles the cleanup of a site's destination before it is built.
|
# Handles the cleanup of a site's destination before it is built.
|
||||||
class Cleaner
|
class Cleaner
|
||||||
|
HIDDEN_FILE_REGEX = /\/\.{1,2}$/
|
||||||
attr_reader :site
|
attr_reader :site
|
||||||
|
|
||||||
def initialize(site)
|
def initialize(site)
|
||||||
|
@ -40,7 +41,7 @@ module Jekyll
|
||||||
dirs = keep_dirs
|
dirs = keep_dirs
|
||||||
|
|
||||||
Dir.glob(site.in_dest_dir("**", "*"), File::FNM_DOTMATCH) do |file|
|
Dir.glob(site.in_dest_dir("**", "*"), File::FNM_DOTMATCH) do |file|
|
||||||
next if file =~ /\/\.{1,2}$/ || file =~ regex || dirs.include?(file)
|
next if file =~ HIDDEN_FILE_REGEX || file =~ regex || dirs.include?(file)
|
||||||
files << file
|
files << file
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue