Skip to content

Conversation

@teojgo
Copy link
Contributor

@teojgo teojgo commented Feb 15, 2023

Fixes #2789

@teojgo teojgo added this to the ReFrame 4.1 milestone Feb 15, 2023
@teojgo teojgo requested review from ekouts and vkarak February 15, 2023 15:12
@teojgo teojgo self-assigned this Feb 15, 2023
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.

I suggest opening this PR against master, as it's not a new feature.

@teojgo teojgo changed the base branch from develop to master February 15, 2023 16:37
Signed-off-by: Theofilos Manitaras <manitaras@cscs.ch>
@teojgo teojgo force-pushed the enhancement/warn_invalid_mode branch from ff94405 to 8fd32d4 Compare February 15, 2023 16:41
@vkarak vkarak changed the title [feat] Raise warning for invalid mode cli option [enhancement] Raise warning for invalid mode cli option Feb 15, 2023
@codecov-commenter
Copy link

Codecov Report

Base: 86.57% // Head: 86.58% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (a99eeb9) compared to base (ec6acac).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2790   +/-   ##
=======================================
  Coverage   86.57%   86.58%           
=======================================
  Files          60       60           
  Lines       11229    11232    +3     
=======================================
+ Hits         9722     9725    +3     
  Misses       1507     1507           
Impacted Files Coverage Δ
reframe/frontend/cli.py 73.35% <100.00%> (+0.12%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@vkarak vkarak merged commit 77b163c into reframe-hpc:master Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Issue a warning if an execution mode requested with --mode is not found

3 participants