Skip to content
This repository has been archived by the owner on Oct 7, 2022. It is now read-only.

(#138) TimeTrackEntry: description should not be optional #139

Merged
merged 1 commit into from Jan 10, 2018
Merged

Conversation

llorllale
Copy link
Owner

(REF) TimeTrackEntry.description: now returns String instead of
          Optional<String>
(REF) EntrySpec: no longer assuming that user provided 'null' value for
          description
(REF) refactor: XmlTimeTrackEntry, XmlTimeTrackEntryTest

closes #138

(REF) TimeTrackEntry.description: now returns String instead of
      Optional<String>
(REF) EntrySpec: no longer assuming that user provided 'null' value for
      description
(REF) refactor: XmlTimeTrackEntry, XmlTimeTrackEntryTest
@llorllale llorllale added the bug label Jan 10, 2018
@llorllale llorllale added this to the 1.0.0 milestone Jan 10, 2018
@llorllale llorllale self-assigned this Jan 10, 2018
@codecov
Copy link

codecov bot commented Jan 10, 2018

Codecov Report

Merging #139 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #139   +/-   ##
=======================================
  Coverage   90.74%   90.74%           
=======================================
  Files          47       47           
  Lines         681      681           
  Branches       32       32           
=======================================
  Hits          618      618           
  Misses         55       55           
  Partials        8        8
Impacted Files Coverage Δ
.../org/llorllale/youtrack/api/IssueTimeTracking.java 91.83% <100%> (ø) ⬆️
.../org/llorllale/youtrack/api/XmlTimeTrackEntry.java 92.3% <100%> (ø) ⬆️

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 d32cac2...4059f41. Read the comment docs.

@llorllale llorllale merged commit feb6069 into master Jan 10, 2018
@llorllale llorllale deleted the 138 branch January 10, 2018 03:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TimeTrackEntry: description should not be optional
1 participant