-
Notifications
You must be signed in to change notification settings - Fork 3
Add support for the PostBuild to extract and copy over Dataset upgrades #194
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
Add support for the PostBuild to extract and copy over Dataset upgrades #194
Conversation
@IsakNaslundBh can I confirm that this PR would not necessitate any update required to either of the wiki procedures listed on this repository? I don't think there is from what I can see, but if you could cast an eye over and double check that'd be handy. |
As it stands, without any tweaks to the verification, that is the case. Had a thought about if we want to get dataset versioning to be checked though by the Versioning verification. If we want to do that, some more tweaks to code as well as procedure is required. WOuld probably be good with a call! |
@BHoMBot check compliance |
@FraserGreenroyd to confirm, the following checks are now queued:
There are 14 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.
See review here
@IsakNaslundBh just to let you know, I have provided a |
@IsakNaslundBh just to let you know, I have provided a |
@BHoMBot check core |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 38 requests in the queue ahead of you. |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 39 requests in the queue ahead of you. |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 40 requests in the queue ahead of you. |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 41 requests in the queue ahead of you. |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 42 requests in the queue ahead of you. |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 43 requests in the queue ahead of you. |
The check |
4 similar comments
The check |
The check |
The check |
The check |
@BHoMBot check ready-to-merge |
@IsakNaslundBh to confirm, the following checks are now queued:
There are 51 requests in the queue ahead of you. |
NOTE: Depends on
Will compile without, but done in support of BHoM/BHoM_Engine#2845
Issues addressed by this PR
Enables the PostBuild to also copy across Dataset upgrades
Test files
See link in Engine PR
Changelog
Additional comments