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

(#113) Non-final inner classes #116

Merged
merged 3 commits into from Dec 20, 2017
Merged

(#113) Non-final inner classes #116

merged 3 commits into from Dec 20, 2017

Conversation

llorllale
Copy link
Owner

(FIX) Both EntrySpec and IssueSpec are now final

(NEW) IssueSpec.summary() : String
(NEW) IssueSpec.description : Optional<String>
(DEL) IssueSpec.nameValuePairs() : List<NameValuePair>
(NEW) UncheckedUriBuilder.paramIfPresent(String, Optional<String>):
      convenience method that adds a parameter to the URI if the
      value is present
(REF) UncheckedUriBuilder.setParameter(String, String) ->
      param(String, String)
(FIX) Both EntrySpec and IssueSpec are now final
@llorllale llorllale added the bug label Dec 20, 2017
@llorllale llorllale added this to the 1.0.0 milestone Dec 20, 2017
@llorllale llorllale self-assigned this Dec 20, 2017
@codecov
Copy link

codecov bot commented Dec 20, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@80ff440). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #116   +/-   ##
=========================================
  Coverage          ?   89.49%           
=========================================
  Files             ?       49           
  Lines             ?      676           
  Branches          ?       30           
=========================================
  Hits              ?      605           
  Misses            ?       63           
  Partials          ?        8
Impacted Files Coverage Δ
.../org/llorllale/youtrack/api/IssueTimeTracking.java 71.79% <ø> (ø)
...c/main/java/org/llorllale/youtrack/api/Issues.java 91.3% <ø> (ø)

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 80ff440...db4e765. Read the comment docs.

@llorllale llorllale merged commit c3cba93 into master Dec 20, 2017
@llorllale llorllale deleted the 113 branch December 20, 2017 04:16
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.

None yet

1 participant