-
Notifications
You must be signed in to change notification settings - Fork 3
8.2 Deployment #305
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
8.2 Deployment #305
Conversation
Temporarily remove the methods failing versioning. To be readded once a fix for them is in place.
…tored to make the versioning check pass
Revert "updated angle calc" This reverts commit 6ed0b63. update angle calc
…latedMaterialLifeCycleEnvironmentalImpactFactors
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No glitches found throughout the testing, can approve on that basis. However, leaving the final word to @adecler who's definitely more in the weeds
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving based on the following:
- Versioning is working in all UIs :Excel, GH with Rhino 7 & 8
- Versioning check is passing
- The test Runner on Versioning gives no errors on any entry from version 7.3 to 8.1.
- There are some errors from version 7.1. and lower (Versioning errors.txt) but those have been confirmed by @pawelbaran to be known and acceptable.
@BHoMBot check ready-to-merge |
1 similar comment
@BHoMBot check ready-to-merge |
@michaelhoehn to confirm, the following actions are now queued:
There are 119 requests in the queue ahead of you. |
@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: ready-to-merge |
@adecler I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results. |
Versioning_Toolkit
Feature
CameraDevice
custom versioningBug