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

Add additional VSTS actions #1609

Closed
gep13 opened this issue May 17, 2017 · 6 comments
Closed

Add additional VSTS actions #1609

gep13 opened this issue May 17, 2017 · 6 comments
Assignees
Milestone

Comments

@gep13
Copy link
Member

gep13 commented May 17, 2017

VSTS uses Logging Commands to allow build scripts to interact with it:

https://github.com/Microsoft/vsts-tasks/blob/master/docs/authoring/commands.md

These are already used for certain things within the VSTS Provider within Cake.

However, it would appear that there are a number of undocumented Commands that can also be utilised. As an example:

https://github.com/Microsoft/vsts-tasks/blob/a67d378b12399fca682f534ed3bea6d317d554c8/Tests-Legacy/L0/PublishCodeCoverageResults/_suite.ts#L38

The ability to Publish Code Coverage Results.

We should find out what these undocumented Commands are, and provide implementations where we can.

I have an open question with the team:

https://twitter.com/vsreleasemgmt/status/864761672577163264

To try to get some clarity on what is, and isn't, supported.

@patriksvensson patriksvensson changed the title Add addition VSTS actions Add additional VSTS actions May 18, 2017
@agc93
Copy link
Member

agc93 commented May 18, 2017

I might be able to get the same support for publishing test results, just testing it has such a brutal feedback cycle 😐

@gep13
Copy link
Member Author

gep13 commented May 18, 2017

Yip, wasn't limiting this issue to only the publishing of code coverage reports. Ideally, anything that is a Logging Command, and makes sense to be included as part of a Cake run should be included.

@agc93
Copy link
Member

agc93 commented May 18, 2017

Yeah agreed, this one's mostly a fight with Microsoft's not-very-good and constantly-changing documentation 😆 Will try and get a look at this over the weekend.

@louisfischer
Copy link
Contributor

Hi @gep13 and @agc93 I currently have a need to have the same additional TFS Commands added to the core cake library. I have some experience with TFS/VSTS and the "Logging Commands." Do you mind if I try to create a pull request and add the additional commands of PublishCodeCoverage and PublishTestResults? These 2 additional features would fill the gap of having separate Tasks in TFS/VSTS to publish test results and code coverage results. I will just follow the existing way the other TFS commands are being generated.

@gep13
Copy link
Member Author

gep13 commented Mar 31, 2018

@louisfischer I would have no objections to you picking this up. Let us know if you have any questions.

@devlead devlead added this to the v0.29.0 milestone Jun 21, 2018
devlead added a commit that referenced this issue Jun 21, 2018
…nto develop

* louisfischer-feature/gh-1609/additional-vsts-actions:
  Initial implementation of PushTestResults and PublishCodeCoverage
@devlead
Copy link
Member

devlead commented Jun 21, 2018

Fixed by #2109

@devlead devlead closed this as completed Jun 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants