search

BHoM / Grasshopper_UI

Tools for Grasshopper
GNU Lesser General Public License v3.0
16 stars 5 forks source link

Remove unused method causing Build issues #704

Closed IsakNaslundBh closed 4 months ago

IsakNaslundBh commented 4 months ago

Issues addressed by this PR

Closes #703

Simply removing a method causing build errors and that is not used anywhere.

Looking at the method removed, it also looks as if the method has the arguments the wrong way. Could attempt to fix the build issue, and fix the method by re-juggling and introducing some local variables, but as long as all bot checks passes with the method removed, should be safe to just merge.

Test files

Changelog

Additional comments

IsakNaslundBh commented 4 months ago

@BHoMBot check installer

bhombot-ci[bot] commented 4 months ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `installer`
FraserGreenroyd commented 4 months ago

@BHoMBot check versioning @BHoMBot check compliance @BHoMBot check required

bhombot-ci[bot] commented 4 months ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `versioning` - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `branch-compliance` - check `dataset-compliance` - check `copyright-compliance` - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `core` - check `null-handling` - check `serialisation` - check `versioning` - check `installer`
bhombot-ci[bot] commented 4 months 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 4 months 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 4 months ago
The check `versioning` 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 4 months 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 4 months ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 4 months ago
@IsakNaslundBh to confirm, the following actions are now queued: - check `ready-to-merge`