Skip to content

MAGN-8735 Re-enable Visualization tests and move into shared assembly. Update job on CI to target new visualization assembly. 2015#681

Merged
ikeough merged 4 commits intoDynamoDS:Revit2015from
ikeough:MAGN-8735_2015
Oct 12, 2015
Merged

MAGN-8735 Re-enable Visualization tests and move into shared assembly. Update job on CI to target new visualization assembly. 2015#681
ikeough merged 4 commits intoDynamoDS:Revit2015from
ikeough:MAGN-8735_2015

Conversation

@ikeough
Copy link

@ikeough ikeough commented Oct 5, 2015

This is the Revit counterpart to the original PR, DynamoDS/Dynamo#5431

Cherry Picks:

  • Revit2016
  • Revit2017

Copy link
Member

Choose a reason for hiding this comment

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

I know this is unrleated- but isn't this exception incorrect should it say ("An instance of RevitDynamoModel is required to construct a DynamoRevitViewModel.");

?

Ian Keough added 2 commits October 12, 2015 14:06
Conflicts:
	src/DynamoRevit/ViewModel/DynamoRevitViewModel.cs
ikeough pushed a commit that referenced this pull request Oct 12, 2015
MAGN-8735 Re-enable Visualization tests and move into shared assembly. Update job on CI to target new visualization assembly. 2015
@ikeough ikeough merged commit f1b8083 into DynamoDS:Revit2015 Oct 12, 2015
@ikeough ikeough deleted the MAGN-8735_2015 branch October 12, 2015 21:09
This was referenced Oct 12, 2015
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