Skip to content

Conversation

@jgphpc
Copy link
Contributor

@jgphpc jgphpc commented Oct 11, 2019

@pep8speaks
Copy link

pep8speaks commented Oct 11, 2019

Hello @jgphpc, Thank you for updating!

Cheers! There are no PEP8 issues in this Pull Request!Do see the ReFrame Coding Style Guide

Comment last updated at 2019-10-16 14:40:12 UTC

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.

I like the idea, but I'm not sure for the tag name yet.

@jgphpc
Copy link
Contributor Author

jgphpc commented Oct 11, 2019

To be clear, i will also add the craype tag but in another PR.

@vkarak vkarak added this to the ReFrame Sprint 2019w41 milestone Oct 11, 2019
@vkarak
Copy link
Contributor

vkarak commented Oct 15, 2019

@jgphpc Rename the tag to external-resources and then we can merge it.

@vkarak vkarak changed the title [test] add resources tag to checks that use the resources dir [test] Tag checks that use external resources (resourcesdir) Oct 16, 2019
@vkarak vkarak changed the title [test] Tag checks that use external resources (resourcesdir) [test] Tag checks that use external resources Oct 16, 2019
@vkarak
Copy link
Contributor

vkarak commented Oct 16, 2019

@jenkins-cscs retry none

@codecov-io
Copy link

codecov-io commented Oct 16, 2019

Codecov Report

Merging #964 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #964      +/-   ##
==========================================
- Coverage   91.83%   91.82%   -0.02%     
==========================================
  Files          80       80              
  Lines       10674    10674              
==========================================
- Hits         9803     9801       -2     
- Misses        871      873       +2
Impacted Files Coverage Δ
reframe/core/config.py 82.9% <0%> (-1.71%) ⬇️

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 e8f3b31...92b51c8. Read the comment docs.

@jgphpc jgphpc merged commit 5aaf2ce into reframe-hpc:master Oct 16, 2019
@jgphpc jgphpc deleted the UES-572_tags branch October 16, 2019 16:44
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.

4 participants