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

Merge stable 1.9 into stable 1.10#789

Merged
szyrom merged 5 commits intopmem:stable-1.10from
lukaszstolarczuk:merge-stable-1.9-into-stable-1.10
Jun 18, 2020
Merged

Merge stable 1.9 into stable 1.10#789
szyrom merged 5 commits intopmem:stable-1.10from
lukaszstolarczuk:merge-stable-1.9-into-stable-1.10

Conversation

@lukaszstolarczuk
Copy link
Copy Markdown
Member

@lukaszstolarczuk lukaszstolarczuk commented Jun 18, 2020

This change is Reviewable

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 18, 2020

Codecov Report

Merging #789 into stable-1.10 will decrease coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           stable-1.10     #789      +/-   ##
===============================================
- Coverage        71.64%   71.51%   -0.14%     
===============================================
  Files               42       42              
  Lines             4493     4493              
===============================================
- Hits              3219     3213       -6     
- Misses            1274     1280       +6     
Flag Coverage Δ
#tests_clang_debug_cpp17 55.52% <ø> (+0.20%) ⬆️
#tests_gcc_debug 69.74% <ø> (-0.01%) ⬇️
Impacted Files Coverage Δ
...ude/libpmemobj++/container/concurrent_hash_map.hpp 86.53% <0.00%> (-0.42%) ⬇️
...j++/container/detail/concurrent_skip_list_impl.hpp 32.35% <0.00%> (-0.31%) ⬇️

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 83f6c2a...c5a2f29. Read the comment docs.

Copy link
Copy Markdown
Contributor

@KFilipek KFilipek left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r3, 1 of 1 files at r5.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lukaszstolarczuk)


.github/workflows/gha.yml, line 1 at r5 (raw file):

#

Any license?

Copy link
Copy Markdown
Contributor

@igchor igchor left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lukaszstolarczuk)

Copy link
Copy Markdown

@karczex karczex left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lukaszstolarczuk)

@szyrom szyrom merged commit a1b9685 into pmem:stable-1.10 Jun 18, 2020
@lukaszstolarczuk lukaszstolarczuk deleted the merge-stable-1.9-into-stable-1.10 branch June 18, 2020 12:28
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.

5 participants