Skip to content
This repository was archived by the owner on Mar 22, 2023. It is now read-only.

Introduce options to enable array/vector/string #270

Merged
merged 1 commit into from
Mar 14, 2019

Conversation

igchor
Copy link
Contributor

@igchor igchor commented Mar 14, 2019

This change is Reviewable

@igchor igchor changed the base branch from master to stable-1.6 March 14, 2019 14:04
@pmem-bot
Copy link
Contributor

@pbalcer and @lukaszstolarczuk please review this pull request

@igchor igchor requested a review from marcinslusarz March 14, 2019 14:09
@codecov
Copy link

codecov bot commented Mar 14, 2019

Codecov Report

Merging #270 into stable-1.6 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           stable-1.6    #270   +/-   ##
==========================================
  Coverage        96.5%   96.5%           
==========================================
  Files              30      30           
  Lines            2091    2091           
==========================================
  Hits             2018    2018           
  Misses             73      73
Flag Coverage Δ
#tests_clang_debug_cpp17 96.93% <ø> (ø) ⬆️
#tests_gcc_debug 97.88% <ø> (ø) ⬆️
#tests_gcc_release_cpp17_no_valgrind ?

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 f93a0d7...a7443f9. Read the comment docs.

Copy link

@szyrom szyrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 3 of 3 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @marcinslusarz)

@igchor igchor force-pushed the add_option_to_enable_feature branch from 5f056a0 to a7443f9 Compare March 14, 2019 15:17
@szyrom szyrom merged commit 4523aa7 into pmem:stable-1.6 Mar 14, 2019
@igchor igchor deleted the add_option_to_enable_feature branch April 19, 2019 13:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants