Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ctest: add ARGS option to custom 'make check' #2252

Merged
merged 1 commit into from
Sep 12, 2018

Conversation

junghans
Copy link
Member

The ARGS make variable is the default for the common ctest calls,
e.g. make test ARGS="-E some_test". This change adds the same
functionality to the custom "make check" of this project.

Related to question in #2250 (comment)

The ARGS make variable is the default for the common ctest calls,
e.g. make test ARGS="-E some_test". This change adds the same
functionality to the custom "make check" of this project.
@codecov
Copy link

codecov bot commented Sep 11, 2018

Codecov Report

Merging #2252 into python will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           python   #2252   +/-   ##
======================================
  Coverage      71%     71%           
======================================
  Files         377     377           
  Lines       18941   18941           
======================================
  Hits        13580   13580           
  Misses       5361    5361

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 7ac9db4...d5f136a. Read the comment docs.

@fweik fweik self-requested a review September 12, 2018 10:43
@fweik fweik merged commit c458034 into espressomd:python Sep 12, 2018
@fweik fweik added this to the Espresso 4.0.1 milestone Sep 13, 2018
RudolfWeeber pushed a commit to RudolfWeeber/espresso that referenced this pull request Oct 15, 2018
ctest: add ARGS option to custom 'make check'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants