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

Use VS 2016 as a temp workaround #1079

Closed
wants to merge 1 commit into from
Closed

Use VS 2016 as a temp workaround #1079

wants to merge 1 commit into from

Conversation

igchor
Copy link
Contributor

@igchor igchor commented Jun 4, 2021

for: actions/runner-images#3475


This change is Reviewable

Copy link
Member

@lukaszstolarczuk lukaszstolarczuk 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 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @igchor)

@codecov
Copy link

codecov bot commented Jun 4, 2021

Codecov Report

Merging #1079 (424eda1) into master (7b9e9f0) will not change coverage.
The diff coverage is n/a.

❗ Current head 424eda1 differs from pull request most recent head 6266815. Consider uploading reports for the commit 6266815 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1079   +/-   ##
=======================================
  Coverage   94.03%   94.03%           
=======================================
  Files          49       49           
  Lines        4726     4726           
=======================================
  Hits         4444     4444           
  Misses        282      282           
Flag Coverage Δ
tests_clang_debug_cpp17 93.49% <ø> (+0.08%) ⬆️
tests_gcc_debug 91.49% <ø> (-0.26%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
include/libpmemobj++/condition_variable.hpp 73.80% <0.00%> (-2.39%) ⬇️
...ude/libpmemobj++/container/concurrent_hash_map.hpp 94.35% <0.00%> (+0.15%) ⬆️

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 7b9e9f0...6266815. Read the comment docs.

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.

2 participants