BHoM / SAP2000_Toolkit

GNU Lesser General Public License v3.0
9 stars 3 forks source link

Added null handling to eliminate failed push bug on empty tag #273

Closed enarhi closed 2 years ago

enarhi commented 2 years ago

NOTE: Depends on

Issues addressed by this PR

Closes #272

Adds null handling to tag string check to avoid error when processing tags if one of them is null.

Test files

Test File

enarhi commented 2 years ago

@BHoMBot check compliance

bhombot-ci[bot] commented 2 years ago
@enarhi to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance`
IsakNaslundBh commented 2 years ago

@BHoMBot check required

bhombot-ci[bot] commented 2 years ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `core` - check `null-handling` - check `serialisation` - check `versioning` - check `installer`
bhombot-ci[bot] commented 2 years ago
The check `code-compliance` has already been run previously and recorded as a successful check. This check has not been run again at this time.
bhombot-ci[bot] commented 2 years ago
The check `documentation-compliance` has already been run previously and recorded as a successful check. This check has not been run again at this time.
bhombot-ci[bot] commented 2 years ago
The check `installer` has already been run previously and recorded as a successful check. This check has not been run again at this time.
IsakNaslundBh commented 2 years ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 2 years ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `ready-to-merge` There are 4 requests in the queue ahead of you.