Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update Power part #3

Closed
wants to merge 1 commit into from
Closed

update Power part #3

wants to merge 1 commit into from

Conversation

HKhademian
Copy link

in this commits:

  • add pow function for all Number types returning their same number type
  • add Number.pow function that accepts all type of numbers and return Double
  • makes pow function infix so that functions can use 1 pow 2 or 3.pow(4)
  • pow now accepts all types of Numbers like 3.5 pow 2 or 3f pow 8 or 8.pow(6L)

in this commits:
* add `pow` function for all Number types returning their same number type
* add `Number.pow` function that accepts all type of numbers and return `Double`
* makes `pow` function `infix` so that functions can use `1 pow 2` or `3.pow(4)`
* `pow` now accepts all type of `Number`s like `3.5 pow 2` or `3f pow 8`
@codecov
Copy link

codecov bot commented Jan 9, 2018

Codecov Report

Merging #3 into master will decrease coverage by 8%.
The diff coverage is n/a.

Impacted file tree graph

@@         Coverage Diff          @@
##           master    #3   +/-   ##
====================================
- Coverage     100%   92%   -8%     
====================================
  Files           9    10    +1     
  Lines          68    75    +7     
  Branches       36    36           
====================================
+ Hits           68    69    +1     
- Misses          0     6    +6
Impacted Files Coverage Δ
src/main/java/com/marcinmoskala/math/Power.kt 14.28% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2209025...86cca95. Read the comment docs.

@MarcinMoskala
Copy link
Owner

As code coverage tool shouts - these methods should be tested.

@HKhademian
Copy link
Author

it errors because in my commit says Int.power(Int):Int is removed but I add Int.power(Number):Int that accepts all the older one with some extra. even their function code is absolutely the same.

@MarcinMoskala
Copy link
Owner

It also shows that test coverage dropped from 100% to 92%

@LarsArtmann
Copy link
Contributor

See pull request #5

MarcinMoskala added a commit that referenced this pull request Jan 12, 2019
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.

None yet

3 participants