Skip to content

Conversation

@jgphpc
Copy link
Contributor

@jgphpc jgphpc commented Jul 29, 2019

Will fix 484

@pep8speaks
Copy link

pep8speaks commented Jul 29, 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-07-29 13:18:59 UTC

@jgphpc jgphpc changed the title inspector [test] inspector Jul 29, 2019
@jgphpc jgphpc requested a review from kraushm July 29, 2019 13:09
@codecov-io
Copy link

codecov-io commented Jul 29, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #890   +/-   ##
=======================================
  Coverage   91.72%   91.72%           
=======================================
  Files          78       78           
  Lines       10308    10308           
=======================================
  Hits         9455     9455           
  Misses        853      853

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 e6b8e11...905d014. Read the comment docs.

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

@jgphpc jgphpc merged commit 180b346 into reframe-hpc:master Jul 29, 2019
@jgphpc jgphpc deleted the UES-484 branch July 29, 2019 13:33
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