-
Notifications
You must be signed in to change notification settings - Fork 4
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
7.1 Deployment #374
7.1 Deployment #374
Conversation
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.
Tested with BHoM_7.1.240312.beta.0
Checked with BHoM_7.1.240315.beta.0 after bug fix merged.
Test was carried out following test procedures.
Happy to approve.
@BHoMBot check compliance |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 89 requests in the queue ahead of you. |
…ince been replaced in Structure_Engine
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.
This can be approved based on @KalleEdstroem's testing. My changes were purely compliance and made no changes to the functionality of the toolkit.
@BHoMBot check compliance |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 59 requests in the queue ahead of you. |
@bhombot-ci[bot] just to let you know, I have provided a |
1 similar comment
@bhombot-ci[bot] just to let you know, I have provided a |
@bhombot-ci[bot] just to let you know, I have provided a |
@bhombot-ci[bot] just to let you know, I have provided a |
@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: null-handling, serialisation, dataset-compliance, unit-tests |
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results. |
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.
PRs have been reviewed by DevOps and bot checks and are suitable for deployment for updating copyright headers to 2024. Approving for merge.
@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: core |
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results. |
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.
PRs have been reviewed by DevOps and are suitable for the deployment of develop
to the 7.1 beta. Approving for merge.
@bhombot-ci[bot] just to let you know, I have provided a |
MidasCivil_Toolkit
Bug
Compliance
Documentation