Skip to content

Solving the Kata#139

Open
sphinix27 wants to merge 1 commit intodevelopfrom
abel_conversion
Open

Solving the Kata#139
sphinix27 wants to merge 1 commit intodevelopfrom
abel_conversion

Conversation

@sphinix27
Copy link
Contributor

Description:

Create a function taking a positive integer as its parameter and returning a string containing the Roman Numeral representation of that integer.

Modern Roman numerals are written by expressing each digit separately starting with the left most digit and skipping any digit with a value of zero. In Roman numerals 1990 is rendered: 1000=M, 900=CM, 90=XC; resulting in MCMXC. 2008 is written as 2000=MM, 8=VIII; or MMVIII. 1666 uses each Roman symbol in descending order: MDCLXVI.

Example:

conversion.solution(1000); //should return "M"

@codecov-io
Copy link

codecov-io commented Sep 4, 2017

Codecov Report

Merging #139 into develop will increase coverage by 0.37%.
The diff coverage is 97.5%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #139      +/-   ##
=============================================
+ Coverage      87.84%   88.21%   +0.37%     
- Complexity       456      463       +7     
=============================================
  Files            103      104       +1     
  Lines            987     1027      +40     
  Branches         161      165       +4     
=============================================
+ Hits             867      906      +39     
  Misses           106      106              
- Partials          14       15       +1
Impacted Files Coverage Δ Complexity Δ
...java/org/fundacionjala/coding/abel/Conversion.java 97.5% <97.5%> (ø) 7 <7> (?)

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 ba1f59d...7875443. Read the comment docs.

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.

2 participants