Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@troels-im
Copy link
Contributor

No description provided.

@troels-im
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@troels-im troels-im marked this pull request as ready for review August 4, 2021 13:29
Copy link
Contributor

@swernli swernli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the manage runci invocation is not actually finding the tests to run. From the logs:

*********************************
No test configuration file found!
*********************************
Usage

  ctest [options]

This might have to do with the current working directory when ctest is invoked, as ctest needs to be run while the working directory is the cmake build output folder.

EDIT: Looking at the other PRs, it may be that this behavior is expected for now. If so, my apologies! I'll update the review accordingly.

@troels-im
Copy link
Contributor Author

It looks like the manage runci invocation is not actually finding the tests to run. From the logs:

*********************************
No test configuration file found!
*********************************
Usage

  ctest [options]

This might have to do with the current working directory when ctest is invoked, as ctest needs to be run while the working directory is the cmake build output folder.

EDIT: Looking at the other PRs, it may be that this behavior is expected for now. If so, my apologies! I'll update the review accordingly.

Indeed there are no tests yet and this is the reason for the output. That is, everything is working as expected.

@troels-im troels-im enabled auto-merge (squash) August 5, 2021 18:31
@swernli
Copy link
Contributor

swernli commented Aug 6, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s), but failed to run 1 pipeline(s).

@troels-im troels-im merged commit e90dfa8 into microsoft:features/llvm-passes Aug 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants