Skip to content

Commit 3b14ae7

Browse files
committed
feat(line): remove default, and use meaningful variable name
1 parent b8f784b commit 3b14ae7

1 file changed

Lines changed: 11 additions & 11 deletions

File tree

lib/git_diff_parser/line.rb

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,17 +7,17 @@ class Line
77
# @!attribute [r] patch_position
88
# @return [Integer] line patch position
99

10-
# @param options [Hash] this is not options
11-
# @option options [Integer] :number line number (required)
12-
# @option options [String] :content content (required)
13-
# @option options [Integer] :patch_position patch position (required)
14-
def initialize(options = {})
15-
fail(ArgumentError('number is required')) unless options[:number]
16-
fail(ArgumentError('content is required')) unless options[:content]
17-
fail(ArgumentError('patch_position is required')) unless options[:patch_position]
18-
@number = options[:number]
19-
@content = options[:content]
20-
@patch_position = options[:patch_position]
10+
# @param params [Hash] required params
11+
# @option params [Integer] :number line number (required)
12+
# @option params [String] :content content (required)
13+
# @option params [Integer] :patch_position patch position (required)
14+
def initialize(params)
15+
fail(ArgumentError('number is required')) unless params[:number]
16+
fail(ArgumentError('content is required')) unless params[:content]
17+
fail(ArgumentError('patch_position is required')) unless params[:patch_position]
18+
@number = params[:number]
19+
@content = params[:content]
20+
@patch_position = params[:patch_position]
2121
end
2222

2323
# @return [Boolean] true if line changed

0 commit comments

Comments
 (0)