1
94%
master: 94%

Ran 08 Aug 2018 11:13AM UTC

Files 91

Run time 5s

Badge

Embed ▾

Committed 08 Aug 2018 11:08AM UTC coverage: 92.168% (-0.02%) from 92.189%

Job # 4609.1

Build Type

push

travis-ci

Committed by

pyokagan

Commit Message

ParserUtil: remove `Optional` variants of parse methods

As the comment[1] at the top of ParserUtil says, methods that take
`Optional` as a parameter are bad.

So, let's remove them.

(It could be argued that leaving these methods in the code base could
serve as an example for students of what not to do, but really, (1) the
number of things that are forbidden under "best practice" is quite
numerous, and trying to add an example of every single one of them in
our code base would just lead to an annoyingly smelly code base, and (2)
students will need to spend time cleaning up all of these negative
examples, which is an unnecessary burden. These things would be better
documented in style guides.)

[1] Comment added in 84d6516f ([#557] ParserUtil: Update header comments
    on Optional usage (#558), 2017-07-18)

Run Details

1424 of 1545 relevant lines covered (92.17%)

0.92 hits per line

se-edu / addressbook-level4 / 4609 / 1
94%
master: 94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4609.1

se-edu / addressbook-level4 / 4609 / 1 94% master: 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4609.1

se-edu / addressbook-level4 / 4609 / 1
94%
master: 94%

README BADGES
x