-
Notifications
You must be signed in to change notification settings - Fork 13
BHoM_Engine: Overall compliance fixes #3472
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
BHoM_Engine: Overall compliance fixes #3472
Conversation
@IsakNaslundBh to confirm, the following actions are now queued:
There are 29 requests in the queue ahead of you. |
Please be advised that the check with reference 39120102283 has more than 50 annotations of notes. API limitations restrict annotations to 50. You may need to rerun this check to obtain the next set when you make changes. At the time of reporting this check, there are 13 additional annotations waiting, made up of 0 errors and 13 warnings. |
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 review
- testing that all code compliance checks are now passing for the BHoM_Engine using
this script
@BHoMBot check required |
@adecler to confirm, the following actions are now queued:
There are 3 requests in the queue ahead of you. |
The check |
The check |
@IsakNaslundBh to confirm, the following actions are now queued:
There are 9 requests in the queue ahead of you. |
The check |
The check |
Issues addressed by this PR
Closes #3451
Aiming to fix all compliance failures throughout the BHoM_Engine.
Test files
Changelog
Additional comments