Skip to content

PHOENIX-6775 Enable code coverage reporting to SonarQube in Phoenix-Omid - #116

Merged
richardantal merged 2 commits into
apache:masterfrom
horvathdora:PHOENIX-6775
Aug 30, 2022
Merged

PHOENIX-6775 Enable code coverage reporting to SonarQube in Phoenix-Omid#116
richardantal merged 2 commits into
apache:masterfrom
horvathdora:PHOENIX-6775

Conversation

@horvathdora

Copy link
Copy Markdown
Contributor

No description provided.

@richardantal richardantal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, just some minor stuff in the readme

Comment thread dev/code-coverage/README.md Outdated

The coverage results can be found under `target/clover/index.html` and here is how you can run the clover code analysis:

```sh dev-support/code-coverage/run-coverage.sh```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this location should be dev/code-coverage/run-coverage.sh

Comment thread dev/code-coverage/README.md Outdated

Here is an example command for running and publishing the coverage data:

`sh dev-support/code-coverage/run-coverage.sh -l ProjectCredentials -u https://exampleserver.com

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

dev/code-coverage/run-coverage.sh here as well.

@richardantal richardantal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@richardantal

Copy link
Copy Markdown
Contributor

Thank you Dora for the fix

@richardantal
richardantal merged commit 8da8c01 into apache:master Aug 30, 2022
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