Skip to content

Conversation

@sekelle
Copy link
Contributor

@sekelle sekelle commented Oct 7, 2019

No description provided.

@pep8speaks
Copy link

pep8speaks commented Oct 7, 2019

Hello @sebkelle1, Thank you for updating!

Line 75:1: E303 too many blank lines (3)

Do see the ReFrame Coding Style Guide

Comment last updated at 2019-11-27 18:08:56 UTC

@codecov-io
Copy link

codecov-io commented Oct 7, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #959   +/-   ##
=======================================
  Coverage   91.66%   91.66%           
=======================================
  Files          81       81           
  Lines       11078    11078           
=======================================
  Hits        10155    10155           
  Misses        923      923

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 dd64282...f9fe29c. Read the comment docs.

@vkarak vkarak assigned vkarak and unassigned vkarak Oct 7, 2019
@vkarak vkarak self-requested a review October 7, 2019 16:58
@vkarak vkarak modified the milestone: Upcoming sprint Oct 7, 2019
@vkarak vkarak requested a review from victorusu October 23, 2019 20:57
@vkarak vkarak added this to the ReFrame sprint 2019w44 milestone Oct 31, 2019
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 now

@vkarak
Copy link
Contributor

vkarak commented Nov 27, 2019

@jenkins-cscs retry daint kesch

@vkarak vkarak merged commit 90309de into reframe-hpc:master Nov 27, 2019
@sekelle sekelle deleted the checks/strided_bandwidth branch November 28, 2019 07: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.

4 participants