Skip to content

Conversation

@jjotero
Copy link
Contributor

@jjotero jjotero commented Oct 29, 2021

Closes #2217

@codecov-commenter
Copy link

Codecov Report

Merging #2255 (d42b5a1) into master (37545d2) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2255      +/-   ##
==========================================
- Coverage   86.13%   86.09%   -0.04%     
==========================================
  Files          56       56              
  Lines       10199    10172      -27     
==========================================
- Hits         8785     8758      -27     
  Misses       1414     1414              
Impacted Files Coverage Δ
reframe/core/fixtures.py 100.00% <100.00%> (ø)
reframe/core/meta.py 98.79% <100.00%> (ø)
reframe/core/namespaces.py 96.07% <100.00%> (-0.65%) ⬇️
reframe/core/parameters.py 97.82% <100.00%> (-0.14%) ⬇️
reframe/core/variables.py 96.03% <100.00%> (-0.08%) ⬇️
reframe/frontend/cli.py 76.30% <0.00%> (+0.04%) ⬆️

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 37545d2...d42b5a1. Read the comment docs.

@vkarak vkarak changed the title [feat] Improve namespace injection [refactor] Improve namespace injection Nov 2, 2021
@vkarak vkarak merged commit b33ba87 into reframe-hpc:master Nov 4, 2021
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.

Refactor internal parameter, variable and fixture namespace injection

3 participants