Skip to content

Conversation

@vagisha-nidhi
Copy link
Contributor

What kind of change does this PR introduce?

This PR is related to the issue #589
The PR helps to publish test results for reporting in Azure Pipelines.

Summary

Azure Pipelines provides excellent reporting capabilities that can be leveraged by this PR.
An example report is shown in the image below.
image
A screenshot for code coverage :
image

Does this PR introduce a breaking change?
No

package.json Outdated
"config": {
"commitizen": {
"path": "./node_modules/cz-conventional-changelog"
"path": "./node_modules/cz-conventional-changelog-default-export"
Copy link
Member

Choose a reason for hiding this comment

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

Hey! This looks pretty good but I was wondering if this line is intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jimthedev Hey. Sorry I missed this conversation. This change wasn't intentional and not related to the working I guess.

Copy link
Member

Choose a reason for hiding this comment

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

If you can update it then I can merge.

@jimthedev
Copy link
Member

@vagisha-nidhi did you see my feedback above?

@jimthedev jimthedev force-pushed the PublishToAzurePipelines branch from 9f8822f to 065b95e Compare July 19, 2019 16:49
@jimthedev jimthedev merged commit 5f71dfe into commitizen:master Jul 19, 2019
@commitizen-bot
Copy link

🎉 This PR is included in version 4.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants