Skip to content

Conversation

@teojgo
Copy link
Contributor

@teojgo teojgo commented Jan 25, 2021

No description provided.

Copy link
Contributor

@ekouts ekouts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, we are supposed to get new fixed modules in the February release

@codecov-io
Copy link

codecov-io commented Jan 25, 2021

Codecov Report

Merging #1712 (86614b5) into master (23f896d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1712   +/-   ##
=======================================
  Coverage   87.31%   87.31%           
=======================================
  Files          46       46           
  Lines        7639     7639           
=======================================
  Hits         6670     6670           
  Misses        969      969           

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 23f896d...acdbcfd. Read the comment docs.

@pep8speaks
Copy link

pep8speaks commented Jan 25, 2021

Hello @teojgo, 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 2021-01-25 22:26:25 UTC

@vkarak vkarak merged commit b3654c2 into reframe-hpc:master Jan 25, 2021
@teojgo teojgo deleted the test/cray_variables_cdt2011 branch May 21, 2021 11:59
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.

5 participants