Skip to content

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

Merged
merged 35 commits into from
Jun 25, 2025
Merged

8.2 Deployment #305

merged 35 commits into from
Jun 25, 2025

Conversation

IsakNaslundBh and others added 30 commits March 31, 2025 11:38
Temporarily remove the methods failing versioning. To be readded once a fix for them is in place.
Revert "updated angle calc"

This reverts commit 6ed0b63.

update angle calc
@BHoMBot BHoMBot added the type:feature New capability or enhancement label Jun 18, 2025
Copy link
Member

@pawelbaran pawelbaran left a 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

Copy link
Member

@adecler adecler left a 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
Copy link
Member Author

BHoMBot commented Jun 25, 2025

@BHoMBot check ready-to-merge

1 similar comment
@michaelhoehn
Copy link
Contributor

@BHoMBot check ready-to-merge

Copy link

bhombot-ci bot commented Jun 25, 2025

@michaelhoehn to confirm, the following actions are now queued:

  • check ready-to-merge

There are 119 requests in the queue ahead of you.

@adecler
Copy link
Member

adecler commented Jun 25, 2025

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: ready-to-merge

Copy link

bhombot-ci bot commented Jun 25, 2025

@adecler I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.

@michaelhoehn michaelhoehn merged commit 2e8ba17 into main Jun 25, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature New capability or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants