Merge pull request #354 from olleolleolle/bugfix/351-exclude_tags
Parse options file options into arrays, integers, flags, and other
This commit is contained in:
commit
9150cdc09e
|
@ -23,9 +23,7 @@ module GitHubChangelogGenerator
|
||||||
|
|
||||||
def parse_line!(line)
|
def parse_line!(line)
|
||||||
key_sym, value = extract_pair(line)
|
key_sym, value = extract_pair(line)
|
||||||
value = true if value =~ /^(true|t|yes|y|1)$/i
|
@options[translate_option_name(key_sym)] = convert_value(value, key_sym)
|
||||||
value = false if value =~ /^(false|f|no|n|0)$/i
|
|
||||||
@options[key_sym] = value
|
|
||||||
rescue
|
rescue
|
||||||
raise ParserError, "Config file #{file} is incorrect in line \"#{line.gsub(/[\n\r]+/, '')}\""
|
raise ParserError, "Config file #{file} is incorrect in line \"#{line.gsub(/[\n\r]+/, '')}\""
|
||||||
end
|
end
|
||||||
|
@ -38,5 +36,39 @@ module GitHubChangelogGenerator
|
||||||
key, value = line.split("=", 2)
|
key, value = line.split("=", 2)
|
||||||
[key.sub("-", "_").to_sym, value.gsub(/[\n\r]+/, "")]
|
[key.sub("-", "_").to_sym, value.gsub(/[\n\r]+/, "")]
|
||||||
end
|
end
|
||||||
|
|
||||||
|
KNOWN_ARRAY_KEYS = [:exclude_labels, :include_labels, :bug_labels,
|
||||||
|
:enhancement_labels, :between_tags, :exclude_tags]
|
||||||
|
KNOWN_INTEGER_KEYS = [:max_issues]
|
||||||
|
|
||||||
|
def convert_value(value, key_sym)
|
||||||
|
if KNOWN_ARRAY_KEYS.include?(key_sym)
|
||||||
|
value.split(",")
|
||||||
|
elsif KNOWN_INTEGER_KEYS.include?(key_sym)
|
||||||
|
value.to_i
|
||||||
|
elsif value =~ /^(true|t|yes|y|1)$/i
|
||||||
|
true
|
||||||
|
elsif value =~ /^(false|f|no|n|0)$/i
|
||||||
|
false
|
||||||
|
else
|
||||||
|
value
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
def translate_option_name(key_sym)
|
||||||
|
{
|
||||||
|
bugs_label: :bug_prefix,
|
||||||
|
enhancement_label: :enhancement_prefix,
|
||||||
|
issues_label: :issue_prefix,
|
||||||
|
header_label: :header,
|
||||||
|
front_matter: :frontmatter,
|
||||||
|
pr_label: :merge_prefix,
|
||||||
|
issues_wo_labels: :add_issues_wo_labels,
|
||||||
|
pr_wo_labels: :add_pr_wo_labels,
|
||||||
|
pull_requests: :pulls,
|
||||||
|
filter_by_milestone: :filter_issues_by_milestone,
|
||||||
|
github_api: :github_endpoint
|
||||||
|
}.fetch(key_sym) { key_sym }
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -1 +1,2 @@
|
||||||
exclude-labels=73a91042-da6f-11e5-9335-1040f38d7f90,7adf83b4-da6f-11e5-ae18-1040f38d7f90
|
exclude-labels=73a91042-da6f-11e5-9335-1040f38d7f90,7adf83b4-da6f-11e5-ae18-1040f38d7f90
|
||||||
|
header_label=# My changelog
|
||||||
|
|
|
@ -43,12 +43,18 @@ describe GitHubChangelogGenerator::ParserFile do
|
||||||
params_file: "spec/files/github_changelog_params_327"
|
params_file: "spec/files/github_changelog_params_327"
|
||||||
}
|
}
|
||||||
end
|
end
|
||||||
|
|
||||||
it "reads exclude_labels into an Array" do
|
it "reads exclude_labels into an Array" do
|
||||||
pending("Related with Bug #327.")
|
|
||||||
expect { parse.parse! }.to change { options[:exclude_labels] }
|
expect { parse.parse! }.to change { options[:exclude_labels] }
|
||||||
.from(nil)
|
.from(nil)
|
||||||
.to(["73a91042-da6f-11e5-9335-1040f38d7f90", "7adf83b4-da6f-11e5-ae18-1040f38d7f90"])
|
.to(["73a91042-da6f-11e5-9335-1040f38d7f90", "7adf83b4-da6f-11e5-ae18-1040f38d7f90"])
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it "translates given header_label into the :header option" do
|
||||||
|
expect { parse.parse! }.to change { options[:header] }
|
||||||
|
.from(nil)
|
||||||
|
.to("# My changelog")
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue
Block a user