BHoM / Python_Toolkit

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

Added version check to existing environment query #126

Closed tg359 closed 11 months ago

tg359 commented 11 months ago

Issues addressed by this PR

Closes #125

Removed all mention of C:\ProgramData in favour of environment variables where possible, and added means of checking existing virtual Python env versions against a target version number.

Test files

Changelog

Additional comments

FraserGreenroyd commented 11 months ago

@BHoMBot check compliance @BHoMBot check required

bhombot-ci[bot] commented 11 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` - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `core` - check `null-handling` - check `serialisation` - check `versioning` - check `installer` There are 16 requests in the queue ahead of you.
bhombot-ci[bot] commented 11 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 11 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 11 months ago
The check `project-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 11 months ago
@tg359 just to let you know, I have provided a `check-installer` result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @FraserGreenroyd on LadybugTools_Toolkit
bhombot-ci[bot] commented 11 months ago
@tg359 just to let you know, I have provided a `check-installer` result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @FraserGreenroyd on LadybugTools_Toolkit
bhombot-ci[bot] commented 11 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.
FraserGreenroyd commented 11 months ago

@BHoMBot check compliance

bhombot-ci[bot] commented 11 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 11 months ago

@BHoMBot check core

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

@BHoMBot check versioning

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

@BHoMBot check null-handling @BHoMBot check serialisation

bhombot-ci[bot] commented 11 months ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `null-handling` - check `serialisation`
FraserGreenroyd commented 11 months ago

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: installer

bhombot-ci[bot] commented 11 months ago
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.
FraserGreenroyd commented 11 months ago

@BHoMBot check ready-to-merge

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

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: ready-to-merge

bhombot-ci[bot] commented 11 months ago
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.