-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fixing shade push #203
Fixing shade push #203
Conversation
@BHoMBot Check required |
@kprusicka to confirm, the following actions are now queued:
|
Thank you @BHoMBot |
@kprusicka you're very welcome! 😄 |
The check |
@BHoMBot check null-handling |
@FraserGreenroyd to confirm, the following actions are now queued:
|
@FraserGreenroyd to confirm, the following actions are now queued:
|
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.
Code changes address the issue, tested through with @kprusicka and null handling added suitably.
Merging for deployment via alphas.
@BHoMBot check ready-to-merge |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 15 requests in the queue ahead of you. |
The check |
The check |
The check |
The check |
@BHoMBot check versioning |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 17 requests in the queue ahead of you. |
@BHoMBot check ready-to-merge |
@FraserGreenroyd to confirm, the following actions are now queued:
|
Issues addressed by this PR
Closes #202