Skip to content

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented Sep 21, 2019

  • The reframe_module configuration option is now ignored; a warning message is issued.
  • It is replaced by the -u|--unload-module MOD option.

Fixes #821.

- Use of 'reframe_module' configuration option is now ignored; a warning message
  is issued.
- It is replaced by the `-u|--unload-module MOD` option.
@codecov-io
Copy link

Codecov Report

Merging #946 into master will decrease coverage by 0.07%.
The diff coverage is 55.55%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #946      +/-   ##
==========================================
- Coverage   91.71%   91.64%   -0.08%     
==========================================
  Files          79       79              
  Lines       10465    10482      +17     
==========================================
+ Hits         9598     9606       +8     
- Misses        867      876       +9
Impacted Files Coverage Δ
reframe/settings.py 100% <ø> (ø) ⬆️
unittests/resources/settings.py 100% <ø> (ø) ⬆️
unittests/test_cli.py 93.27% <33.33%> (-2.89%) ⬇️
reframe/frontend/cli.py 80.25% <83.33%> (-0.07%) ⬇️
reframe/core/modules.py 66.76% <0%> (+0.58%) ⬆️

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 58428e9...56df453. Read the comment docs.

@vkarak vkarak merged commit 7d6504e into reframe-hpc:master Sep 24, 2019
@vkarak vkarak deleted the feat/unload-reframe-module branch September 24, 2019 13:40
yqshao added a commit to c3se/hpc2n-reframe-tests that referenced this pull request Mar 10, 2025
`reframe_module` has been deprecated as of:
reframe-hpc/reframe#946
yqshao added a commit to c3se/hpc2n-reframe-tests that referenced this pull request Mar 10, 2025
`reframe_module` has been deprecated as of:
reframe-hpc/reframe#946
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.

Allow setting different reframe_module per system or system partition

3 participants