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

Updating default configurations for debug and release #417

Merged
merged 1 commit into from
Oct 21, 2019

Conversation

al-fisher
Copy link
Member

Issues addressed by this PR

Closes #416

Following update of NuGets - have aligned the default debug and release to debug2018 and release2018 configurations to ensure any default set ups can also build.

@al-fisher al-fisher added severity:critical No workaround exists. Essential to continue type:bug Error or unexpected behaviour labels Oct 19, 2019
@al-fisher al-fisher self-assigned this Oct 19, 2019
Copy link
Contributor

@FraserGreenroyd FraserGreenroyd left a comment

Choose a reason for hiding this comment

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

I have managed to build through all configurations from default (Debug and Release) up to the 2018-2020 configurations.

I have managed to successfully pull a model from Revit into Rhino through the Revit Adapter.

I am approving this PR on the basis that I have managed to achieve the above. The model I pulled through was an Environmental model though, so I thoroughly recommend awaiting a review from @IsakNaslundBh prior to merging 😄

Copy link
Contributor

@IsakNaslundBh IsakNaslundBh left a comment

Choose a reason for hiding this comment

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

Also able to build to all configurations. Also tested pulling structural model which works. APproved

@al-fisher
Copy link
Member Author

Fantastic - thanks both @FraserGreenroyd @IsakNaslundBh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity:critical No workaround exists. Essential to continue type:bug Error or unexpected behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure default (debug and release) configurations build
3 participants