-
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
6.1 Deployment #150
6.1 Deployment #150
Conversation
@BHoMBot check beta-merge-status |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 15 requests in the queue ahead of you. |
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 code changes.
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.
Approved based on Testing Procedure on the BHoM_6.1.230315.beta.0 installer.
@BHoMBot check beta-merge-status |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 15 requests in the queue ahead of you. |
The check |
Re-approving based on the testing with BHoM_6.1.230320.beta.0 and test procedure. All sections returned a pass. |
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.
I have reviewed the code and all looks fine.
I am happy with the test procedure used by @alelom .
I have reviewed the dependencies around this toolkit and am happy this will not negatively impact anything.
I am happy for this to be deployed in the 6.1 beta.
@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 |
2 similar comments
@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 check compliance |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 25 requests in the queue ahead of you. |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 260 requests in the queue ahead of you. |
@BHoMBot check serialisation |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 361 requests in the queue ahead of you. |
@BHoMBot check ready-to-merge |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 434 requests in the queue ahead of you. |
@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: code-compliance, ready-to-merge, documentation-compliance |
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results. |
File_Toolkit
Compliance