Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Update documentation for testing #19063

Merged

Conversation

AaronRobinsonMSFT
Copy link
Member

No description provided.

@AaronRobinsonMSFT AaronRobinsonMSFT added documentation Documentation bug or enhancement, does not impact product or test code test enhancement Improvements of test source code labels Jul 20, 2018
@AaronRobinsonMSFT
Copy link
Member Author

see #18594

@AaronRobinsonMSFT
Copy link
Member Author

cc @sbomer @chsienki


**!! Note !! -- Passing /p:__BuildOs=[OSX|Linux] is required.**

>If you omit it you will get the following error: `error MSB4801: The task factory "CodeTaskFactory" could not be loaded because this version of MSBuild does not support it.`
---
Omitting the above will resulting in the following error: `error MSB4801: The task factory "CodeTaskFactory" could not be loaded because this version of MSBuild does not support it.`

Choose a reason for hiding this comment

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

This is not true as of #19044 (I removed CodeTaskFactory)

Choose a reason for hiding this comment

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

Although users still need to set BuildOS so it ends up in the correct location

Copy link
Member Author

Choose a reason for hiding this comment

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

There is literally not enough +1's to describe how good that is to hear. Thanks!

@AaronRobinsonMSFT AaronRobinsonMSFT merged commit 2556ad9 into dotnet:master Jul 20, 2018
@AaronRobinsonMSFT AaronRobinsonMSFT deleted the merge_more_test_doc branch July 24, 2018 01:46
jashook pushed a commit to jashook/coreclr that referenced this pull request Aug 14, 2018
* Update documentation for testing
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Documentation bug or enhancement, does not impact product or test code test enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants