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

10864 multiple projects issue #1774

Merged
merged 9 commits into from
Nov 11, 2019
Merged

Conversation

ozslab
Copy link
Contributor

@ozslab ozslab commented Nov 7, 2019

Types of changes

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist:

  • I've added tests for my code.
  • I have added an entry to the changelog

Copy link
Contributor

@SabotageAndi SabotageAndi left a comment

Choose a reason for hiding this comment

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

Looks fine for me! Well done!

@ozslab ozslab force-pushed the 10864-multiple-projects-issue branch from 1e670af to 819440a Compare November 8, 2019 14:54
@SabotageAndi SabotageAndi merged commit 7a13b5a into master Nov 11, 2019
@SabotageAndi SabotageAndi deleted the 10864-multiple-projects-issue branch November 11, 2019 09:31
idg10 pushed a commit to idg10/SpecFlow that referenced this pull request Jan 8, 2020
* Make AssemblyHooks internal for NUnit

* Make AssemblyHooks internal for MSTest

* Use prefix in AssemblyHooks for xUnit

* Use $(IntermediateOutputPath) instead of obj folder + Fix the issue of non-existing directory

* Additional changes regarding AssemblyHooks for xUnit:

- Add for VB as well
- Improve targets file

* Use prefix in AssemblyHooks for MSTest

* Use prefix in AssemblyHooks for NUnit

* Extend Specs - Cover the scenario of "Two projects with the same unit test provider"

* Fix: Do not refer to task ReplaceTokenInFileTask if not imported
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.

2 participants