Skip to content

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented May 26, 2020

Fixes #1339.

@vkarak vkarak added this to the ReFrame sprint 20.08 milestone May 26, 2020
@vkarak vkarak requested a review from teojgo May 26, 2020 20:13
@vkarak vkarak self-assigned this May 26, 2020
@codecov-commenter
Copy link

codecov-commenter commented May 26, 2020

Codecov Report

Merging #1340 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1340      +/-   ##
==========================================
+ Coverage   91.73%   91.75%   +0.01%     
==========================================
  Files          83       83              
  Lines       12491    12493       +2     
==========================================
+ Hits        11459    11463       +4     
+ Misses       1032     1030       -2     
Impacted Files Coverage Δ
reframe/core/config.py 88.65% <ø> (+0.56%) ⬆️
unittests/test_config.py 100.00% <100.00%> (ø)
reframe/frontend/cli.py 81.72% <0.00%> (+0.06%) ⬆️

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 372e168...8ec0d8c. Read the comment docs.

@vkarak vkarak requested review from ekouts and victorusu May 27, 2020 16:36
Copy link
Contributor

@victorusu victorusu left a comment

Choose a reason for hiding this comment

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

lgtm. Simple, but with great impact. 😛

@vkarak vkarak merged commit 9ddadeb into reframe-hpc:master May 27, 2020
@vkarak vkarak deleted the bugfix/default-config-value branch May 27, 2020 20:09
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.

Incorrect configuration value returned when a name-addressable object does not exist

3 participants