BHoM / Python_Toolkit

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

Fixed VirtualEnvironmentExists #128

Closed Tom-Kingstone closed 9 months ago

Tom-Kingstone commented 9 months ago

NOTE: Depends on

Issues addressed by this PR

Closes #127

Added a check for if the executable it is trying to run actually exists before running.

Test files

run LBT environment installer in grasshopper.

Changelog

Additional comments

Tom-Kingstone commented 9 months ago

@BHoMBot check compliance @BHoMBot check core

bhombot-ci[bot] commented 9 months ago
@Tom-Kingstone 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` - check `core` There are 19 requests in the queue ahead of you.
Tom-Kingstone commented 9 months ago

@BHoMBot check compliance @BHoMBot check required

bhombot-ci[bot] commented 9 months ago
@Tom-Kingstone 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` - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `core` - check `null-handling` - check `serialisation` - check `versioning` - check `installer`
bhombot-ci[bot] commented 9 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 9 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.
FraserGreenroyd commented 9 months ago

@BHoMBot check compliance

bhombot-ci[bot] commented 9 months ago
@FraserGreenroyd 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`
FraserGreenroyd commented 9 months ago

@BHoMBot check required

bhombot-ci[bot] commented 9 months ago
@FraserGreenroyd 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 9 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 9 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.
FraserGreenroyd commented 9 months ago

@BHoMBot check ready-to-merge

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