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

Send test coverage reports to coveralls #2043

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

sravyap135
Copy link
Contributor

@sravyap135 sravyap135 commented Oct 18, 2021

Description: Coveralls integration with azure

Changes Proposed in PR:

  • updated readme badge for azure pipeline build status
  • added new readme badge for coveralls testcase coverage percent
  • update coveralls push step

Fixes: internal CONTCNTR-2474

Known issues - Broken commit message in coveralls - lemurheavy/coveralls-public#1394

Successful run - https://dev.azure.com/f5networks/CIS/_build/results?buildId=1113&view=logs&j=10cdb312-541c-5bcd-418b-9a3c1da5544f&t=c95c4113-b8c7-5df6-40b2-5f19d93499df&l=1333

https://coveralls.io/builds/43570646

@sravyap135
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

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

@sravyap135
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

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

@sravyap135
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

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

@sravyap135
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

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

@sravyap135
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

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

Copy link
Contributor

@agupta49 agupta49 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@vklohiya vklohiya left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@sravyap135
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

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

@sravyap135 sravyap135 merged commit ecc8893 into F5Networks:master Oct 19, 2021
nandakishorepeddi pushed a commit to nandakishorepeddi/k8s-bigip-ctlr that referenced this pull request Nov 10, 2021
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