Skip to content

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented Jul 7, 2020

Fixes #1382.

@vkarak vkarak added this to the ReFrame sprint 20.10 milestone Jul 7, 2020
@vkarak vkarak requested review from ekouts and victorusu July 7, 2020 11:39
@vkarak vkarak self-assigned this Jul 7, 2020
Copy link
Contributor

@victorusu victorusu 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-commenter
Copy link

codecov-commenter commented Jul 7, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1401   +/-   ##
=======================================
  Coverage   91.77%   91.77%           
=======================================
  Files          83       83           
  Lines       12708    12708           
=======================================
  Hits        11663    11663           
  Misses       1045     1045           
Impacted Files Coverage Δ
reframe/core/decorators.py 96.90% <ø> (ø)

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 072b358...553fb5c. Read the comment docs.

@vkarak vkarak merged commit 95fb217 into reframe-hpc:master Jul 8, 2020
@vkarak vkarak deleted the doc/enhance-require-deps branch July 8, 2020 22:21
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.

Document when '@require_deps'-decorated functions run

4 participants