Skip to content
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

Added get_exe_path with more features #347

Merged
merged 10 commits into from
May 25, 2024
Merged

Added get_exe_path with more features #347

merged 10 commits into from
May 25, 2024

Conversation

bdestombe
Copy link
Collaborator

Use previously installed anywhere. If doesn't exist, download it.

Use previously installed anywhere. If doesn't exist, download it.
@bdestombe
Copy link
Collaborator Author

As I am switching a lot between different Python environments and recreating them, the implemented features greatly reduce the number of having to download the executables.

@bdestombe
Copy link
Collaborator Author

There are two minor remarks on the coding style from codacy, and I don't see easy fixes for those. The try/except should catch multiple errors (haven't encountered all of them) and the other one I don't understand.

@bdestombe
Copy link
Collaborator Author

The tests failing are not related to this PR. They were also failing in the previous commit: #345

nlmod/util.py Outdated Show resolved Hide resolved
nlmod/util.py Outdated Show resolved Hide resolved
nlmod/util.py Outdated Show resolved Hide resolved
@bdestombe bdestombe merged commit 0a6394a into dev May 25, 2024
4 checks passed
@bdestombe bdestombe deleted the get-exe-path branch May 25, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants