will/github-changelog-generator
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #355 from olleolleolle/refactor/name-constant-irregular-options

Browse Source 
Refactor: call it option_name, instead of key_sym
This commit is contained in:
Olle Jonsson 2016-03-19 09:46:09 +01:00
commit f9ad19285e
1 changed files with 22 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
lib/github_changelog_generator/parser_file.rb
View File

@ -22,13 +22,13 @@ module GitHubChangelogGenerator
end
def parse_line!(line)
key_sym, value = extract_pair(line)
@options[translate_option_name(key_sym)] = convert_value(value, key_sym)
option_name, value = extract_pair(line)
@options[option_key_for(option_name)] = convert_value(value, option_name)
rescue
raise ParserError, "Config file #{file} is incorrect in line \"#{line.gsub(/[\n\r]+/, '')}\""
end
# Returns a the setting as a symbol and its string value sans newlines.
# Returns a the option name as a symbol and its string value sans newlines.
#
# @param line [String] unparsed line from config file
# @return [Array<Symbol, String>]
@ -41,10 +41,10 @@ module GitHubChangelogGenerator
:enhancement_labels, :between_tags, :exclude_tags]
KNOWN_INTEGER_KEYS = [:max_issues]
def convert_value(value, key_sym)
if KNOWN_ARRAY_KEYS.include?(key_sym)
def convert_value(value, option_name)
if KNOWN_ARRAY_KEYS.include?(option_name)
value.split(",")
elsif KNOWN_INTEGER_KEYS.include?(key_sym)
elsif KNOWN_INTEGER_KEYS.include?(option_name)
value.to_i
elsif value =~ /^(true|t|yes|y|1)$/i
true
@ -55,8 +55,7 @@ module GitHubChangelogGenerator
end
end
def translate_option_name(key_sym)
{
IRREGULAR_OPTIONS = {
bugs_label: :bug_prefix,
enhancement_label: :enhancement_prefix,
issues_label: :issue_prefix,
@ -68,7 +67,10 @@ module GitHubChangelogGenerator
pull_requests: :pulls,
filter_by_milestone: :filter_issues_by_milestone,
github_api: :github_endpoint
}.fetch(key_sym) { key_sym }
}
def option_key_for(option_name)
IRREGULAR_OPTIONS.fetch(option_name) { option_name }
end
end
end