Skip to content

Conversation

@teojgo
Copy link
Contributor

@teojgo teojgo commented May 11, 2020

  • The chosen level of verbosity was not affecting the printer after
    the update of the site configuration.

Fixes #1295

* The chosen level of verbosity was not affecting the printer after
  the update of the site configuration.
@teojgo teojgo added this to the ReFrame sprint 20.07 milestone May 11, 2020
@teojgo teojgo requested a review from vkarak May 11, 2020 08:48
@teojgo teojgo self-assigned this May 11, 2020
@codecov-io
Copy link

codecov-io commented May 11, 2020

Codecov Report

Merging #1307 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1307   +/-   ##
=======================================
  Coverage   91.62%   91.63%           
=======================================
  Files          83       83           
  Lines       12498    12499    +1     
=======================================
+ Hits        11451    11453    +2     
+ Misses       1047     1046    -1     
Impacted Files Coverage Δ
reframe/frontend/cli.py 81.35% <100.00%> (+0.06%) ⬆️
reframe/core/logging.py 82.55% <0.00%> (+0.31%) ⬆️

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 e0f8d96...99cbcee. Read the comment docs.

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

@vkarak vkarak changed the title [bugfix] Fix setting of printer verbosity after config update [bugfix] Fix behaviour of verbosity option May 18, 2020
@vkarak vkarak merged commit 0faf579 into reframe-hpc:master May 18, 2020
@teojgo teojgo deleted the bugfix/verbose_output branch May 25, 2020 12:48
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.

Increasing verbosity does not print stack traces in case of faulty tests

3 participants