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

Remove duplicate Nexus example tests #880

Merged
merged 2 commits into from
Jun 6, 2018

Conversation

jtkrogel
Copy link
Contributor

@jtkrogel jtkrogel commented Jun 6, 2018

Remove ctests that duplicate Nexus user example tests in ntest. Clean and partly modernize the user example tests. Fix failing settings test when called from ctest (attempted double read of command line flags).

@ghost ghost assigned jtkrogel Jun 6, 2018
@ghost ghost added the in progress label Jun 6, 2018
@qmc-robot
Copy link

Can one of the maintainers verify this patch?

1 similar comment
@qmc-robot
Copy link

Can one of the maintainers verify this patch?

@ye-luo
Copy link
Contributor

ye-luo commented Jun 6, 2018

Okay to test

@ye-luo ye-luo merged commit aafd05a into QMCPACK:develop Jun 6, 2018
@ghost ghost removed the in progress label Jun 6, 2018
@jtkrogel jtkrogel deleted the nx_remove_duplicate_tests branch August 30, 2018 18:36
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.

3 participants