Skip to content

Added missing no_single_resource method#190

Merged
etr merged 1 commit intomasterfrom
no_single_resource
Mar 13, 2020
Merged

Added missing no_single_resource method#190
etr merged 1 commit intomasterfrom
no_single_resource

Conversation

@etr
Copy link
Copy Markdown
Owner

@etr etr commented Mar 13, 2020

Identify the Bug

The no_single_resource was missing from the create_webserver object.

See also: #189

Description of the Change

Added the no_single_resource method

Alternate Designs

Given that default is false, we could have left the method out but it might still be useful in complex configurations of the builder object.

Possible Drawbacks

None.

Verification Process

Unit tests

Release Notes

Added the no_single_resource method to create_webserver

@etr etr self-assigned this Mar 13, 2020
@etr etr added the bug Confirmed bugs or reports that are very likely to be bugs. label Mar 13, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 13, 2020

Codecov Report

Merging #190 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #190   +/-   ##
=======================================
  Coverage   95.19%   95.19%           
=======================================
  Files          35       35           
  Lines        3140     3140           
=======================================
  Hits         2989     2989           
  Misses        151      151
Impacted Files Coverage Δ
src/httpserver/create_webserver.hpp 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 f6810d9...98bf321. Read the comment docs.

@etr etr merged commit a6c82e5 into master Mar 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Confirmed bugs or reports that are very likely to be bugs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant