-
Notifications
You must be signed in to change notification settings - Fork 9
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
hunter integration and refactor #11
hunter integration and refactor #11
Conversation
I tried running this in a fresh venv --
|
Similarly to joes issue, I hit this issue when trying to do
|
Just ran orion in a new venv it is working well on my end. Can you please check if the hunter is installed using
|
Trying |
Think we should actually add in the installation of the requirements in each of these python version workflows so we can catch these failures before hand |
Just to update what I did to make this work
|
Signed-off-by: Shashank Reddy Boyapally <[email protected]>
Signed-off-by: Shashank Reddy Boyapally <[email protected]>
32cc08c
to
8e9c9e6
Compare
Can we add to the readme on python versions to use and versions to avoid and why @shashank-boyapally ? |
Signed-off-by: Shashank Reddy Boyapally <[email protected]>
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.
lgtm! Tested Locally!
sorry for being a man in the middle here. I have seen many python repos struggling with dependency management quite a bit when there is only a requirements.txt. In order to get rid of these issues we need to have a dependency management tool integrated to the project. Here is one of them: https://pdm-project.org/latest/, which satisfies all the below asks It should decide how we will manage our list of deps |
Hi @vishnuchalla, thank you for adding this idea. This would relieve us from dependency management. We should add this in a future PR. |
Signed-off-by: Shashank Reddy Boyapally <[email protected]>
Lets home in on Python 3.11 -- not sure we need to worry with older versions of python. |
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.
/lgtm
Signed-off-by: Shashank Reddy Boyapally <[email protected]>
made changes accordingly to support only Python 3.11. Thank you for the insights @jtaleric @paigerube14 |
Type of change
Description
Hunter is being integrated so that change points can be detected. Refactored code to increase readability.
Below is the sample output
Related Tickets & Documents
Checklist before requesting a review
Testing