Skip to content

Conversation

@jgphpc
Copy link
Contributor

@jgphpc jgphpc commented Nov 8, 2019

Will fix UES-606

@vkarak
Copy link
Contributor

vkarak commented Nov 8, 2019

@jgphpc Test is failing on Dom. Is this expected?

@jgphpc
Copy link
Contributor Author

jgphpc commented Nov 9, 2019

@jenkins-cscs retry dom

@pep8speaks
Copy link

pep8speaks commented Nov 9, 2019

Hello @jgphpc, Thank you for updating!

Line 84:31: E201 whitespace after '{'

Do see the ReFrame Coding Style Guide

Comment last updated at 2019-11-11 12:30:31 UTC

@codecov-io
Copy link

codecov-io commented Nov 9, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1040      +/-   ##
==========================================
- Coverage   91.99%   91.97%   -0.02%     
==========================================
  Files          81       81              
  Lines       10937    10937              
==========================================
- Hits        10061    10059       -2     
- Misses        876      878       +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 ec648b2...c5aae8b. Read the comment docs.

@vkarak vkarak merged commit bb9230d into reframe-hpc:master Nov 11, 2019
@jgphpc jgphpc deleted the UES-606_gperf branch November 11, 2019 13:52
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