-
-
Notifications
You must be signed in to change notification settings - Fork 70
"Refactor to Use 'python' Instead of 'python3' for Environment Consistency" #222
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
Conversation
Quality Gate Passed Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.
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.
Good catch.
Confirmed that invoking a virtual environment’s Python should refer to python
, as opposed to python3
.
6d2addc
into
hackersandslackers:master
Pull Request Description
Title: Remove Redundant
python3
Calls and Usepython
InsteadRelated Issue: # Issue Title: Redundant python3 call
Background:
In our current setup, the code is invoking
python3
for executing scripts even after successfully activating the virtual environment. This reliance on the system'spython3
can lead to inconsistencies, particularly if different environments or versions of Python are being used across different systems. By making our project self-contained and fully utilizing thepython
command from the activated environment, we can ensure that the intended version of Python is consistently used, thereby reducing potential errors or conflicts.Summary of Changes:
This pull request implements the following changes to resolve the redundant use of
python3
:Refactored Code Base:
python3
have been replaced withpython
in the relevant scripts and command-line instructions.python
operates correctly in a properly activated virtual environment.Documentation Updates:
python3
topython
.Testing:
python3
withpython
functions correctly without any introduction of issues.Resolution Benefits:
Next Steps:
Thank you for considering this important improvement for our project's maintainability and user experience!