Skip to content

Conversation

@teojgo
Copy link
Contributor

@teojgo teojgo commented Jul 31, 2018

  • Use new build system feature

  • Use new syntax for test

Related to #397

* Use new build system feature

* Use new syntax for test
Copy link
Contributor

@kraushm kraushm left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #414   +/-   ##
=======================================
  Coverage   91.21%   91.21%           
=======================================
  Files          70       70           
  Lines        8582     8582           
=======================================
  Hits         7828     7828           
  Misses        754      754
Impacted Files Coverage Δ
unittests/test_policies.py 99.54% <0%> (ø) ⬆️

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 98fcf1c...ab57d42. Read the comment docs.

Copy link
Contributor

@vkarak vkarak left a comment

Choose a reason for hiding this comment

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

lgtm

@vkarak vkarak merged commit 9d9ba2e into master Jul 31, 2018
@vkarak vkarak deleted the regression_test/nvprof_PE18.07 branch July 31, 2018 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants