Skip to content

Improve options#148

Merged
hansenmc merged 4 commits into
developmentfrom
improveOptions
Mar 15, 2021
Merged

Improve options#148
hansenmc merged 4 commits into
developmentfrom
improveOptions

Conversation

@hansenmc

Copy link
Copy Markdown
Member
  • Enable xcc.httpcompliant, if not explicitly set as System property or CoRB option
  • Consolidate logic for checking System properties and CoRB options and looking for normalized keys in both Snake_Case and Kebab-Case in a convenience method Options.findOption.
  • Ensure that MAX-OPTS-FROM-MODULE works if specified as original name MAX_OPTS_FROM_MODULE
  • updated README.md with consistent formatting and language for option default values, replaced Bintray download badge with Maven Central

…or CoRB option

- Consolidate logic for checking System properties and CoRB options and looking for normalized keys in both Snake_Case and Kebab-Case in a convenience method Options.findOption.
- Ensure that MAX-OPTS-FROM-MODULE works if specified as original name MAX_OPTS_FROM_MODULE
- updated README.md with consistent formatting and language for option default values
@codecov

codecov Bot commented Mar 13, 2021

Copy link
Copy Markdown

Codecov Report

Merging #148 (8d634b5) into development (e4fced8) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                Coverage Diff                @@
##             development     #148      +/-   ##
=================================================
+ Coverage          83.65%   83.70%   +0.04%     
- Complexity          1295     1298       +3     
=================================================
  Files                 45       46       +1     
  Lines               4204     4209       +5     
  Branches             630      629       -1     
=================================================
+ Hits                3517     3523       +6     
  Misses               461      461              
+ Partials             226      225       -1     
Impacted Files Coverage Δ Complexity Δ
.../com/marklogic/developer/corb/AbstractManager.java 90.47% <100.00%> (-0.07%) 100.00 <2.00> (-3.00)
...ava/com/marklogic/developer/corb/AbstractTask.java 83.76% <100.00%> (+0.21%) 79.00 <1.00> (-1.00) ⬆️
...ain/java/com/marklogic/developer/corb/Options.java 100.00% <100.00%> (ø) 6.00 <6.00> (?)
...java/com/marklogic/developer/corb/TaskFactory.java 92.04% <100.00%> (ø) 35.00 <0.00> (ø)

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 e4fced8...8d634b5. Read the comment docs.

@hansenmc
hansenmc merged commit 624b7e9 into development Mar 15, 2021
@hansenmc
hansenmc deleted the improveOptions branch March 15, 2021 00:29
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.

1 participant