Skip to content

Test PR#7

Open
kolosek wants to merge 14 commits into
developmentfrom
master
Open

Test PR#7
kolosek wants to merge 14 commits into
developmentfrom
master

Conversation

@kolosek

@kolosek kolosek commented Nov 4, 2018

Copy link
Copy Markdown
Owner

No description provided.

Comment thread spec/lib/loan_spec.rb

it "should return correct apr value" do
loan = Loan.new(15, 36, 10000, 5, 3, 10)
loan = Loan.new(nominal_rate: 15, duration: 36, amount: 10000, structure_fee: 5, currency_protection: 3, fee: 10)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.
Metrics/LineLength: Line is too long. [119/80]

Comment thread spec/lib/loan_spec.rb

it "should return correct apr value" do
loan = Loan.new(13, 1, 50000000)
loan = Loan.new(nominal_rate: 13, duration: 1, amount: 50000000)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.

Comment thread spec/lib/loan_spec.rb

it "should return correct apr value" do
loan = Loan.new(13, 90, 50000000)
loan = Loan.new(nominal_rate: 13, duration: 90, amount: 50000000)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.

Comment thread spec/lib/loan_spec.rb

it "should return correct apr value" do
loan = Loan.new(13, 90, 10000000)
loan = Loan.new(nominal_rate: 13, duration: 90, amount: 10000000)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.

Comment thread spec/lib/loan_spec.rb

it "should return correct apr value" do
loan = Loan.new(13, 36, 10000)
loan = Loan.new(nominal_rate: 13, duration: 36, amount: 10000)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.

Comment thread spec/lib/loan_spec.rb

it "should return correct pmt value" do
loan = Loan.new(13, 90, 1000000)
loan = Loan.new(nominal_rate: 13, duration: 90, amount: 1000000)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.

Comment thread spec/lib/loan_spec.rb

it "should return correct pmt value" do
loan = Loan.new(0, 6, 10000)
loan = Loan.new(nominal_rate: 0, duration: 6, amount: 10000)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.

Comment thread spec/lib/loan_spec.rb

it "should return correct pmt value" do
loan = Loan.new(0, 36, 10000)
loan = Loan.new(nominal_rate: 0, duration: 36, amount: 10000)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/NumericLiterals: Use underscores(_) as decimal mark and separate every 3 digits with them.

Comment thread lib/version.rb Outdated
@@ -1,3 +1,3 @@
module FinanceMath
VERSION = "1.0.1"
VERSION = "1.0.5"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/MutableConstant: Freeze mutable objects assigned to constants.
Style/StringLiterals: Prefer single-quoted strings when you don't need string interpolation or special symbols.

Comment thread lib/finance_math/loan.rb

private

def initialize_options(options)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Layout/IndentationConsistency: Inconsistent indentation detected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants