Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Xunit2 support WIP #575

Merged
merged 6 commits into from
Oct 30, 2014
Merged

Xunit2 support WIP #575

merged 6 commits into from
Oct 30, 2014

Conversation

mantzas
Copy link

@mantzas mantzas commented Oct 25, 2014

XUnit 2 has a breaking changes
Arguments are now preffixed with "-" instead of "/".
Some argument went away, new ones were added. (parallel , appveyor support)

@forki
Copy link
Member

forki commented Oct 27, 2014

only 2lines diff?

@mantzas
Copy link
Author

mantzas commented Oct 27, 2014

No it is a working base. First version will come soon.

@forki
Copy link
Member

forki commented Oct 27, 2014

Ok. I thought there might be issues with merge or something.
On Oct 27, 2014 10:32 AM, "Sotirios Mantziaris" notifications@github.com
wrote:

No it is a working base. First version will come soon.


Reply to this email directly or view it on GitHub
#575 (comment).

@mantzas
Copy link
Author

mantzas commented Oct 27, 2014

This would be a beta version. Tested the new arguments in my own Teamcity, worked nicely

@@ -0,0 +1,192 @@
[<AutoOpen>]
Copy link
Member

Choose a reason for hiding this comment

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

please remove this. We try to get rid of autoopen

@mantzas
Copy link
Author

mantzas commented Oct 30, 2014

is this ok for final approval? Anything else to do?

@forki
Copy link
Member

forki commented Oct 30, 2014

Didn't see last commit. Will check

forki added a commit that referenced this pull request Oct 30, 2014
@forki forki merged commit 7f4c281 into fsprojects:master Oct 30, 2014
@forki
Copy link
Member

forki commented Oct 30, 2014

release in 3.8.0

happy testing

happy

@mantzas
Copy link
Author

mantzas commented Oct 30, 2014

thanks

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants