-
Notifications
You must be signed in to change notification settings - Fork 13
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
Try to fix codecov #30
Conversation
This reverts commit 642a087.
@selimsami I have added a temporal fix to the codecov where I had qforce/.github/workflows/CI.yaml Line 52 in 0fa5057
changed to python -m pip install -e . I have no idea why it won't work without the -e but at least the code coverage is working fine now.
|
Hi @xiki-tempula , I made some changes to the CI implementation and managed to fix the requirement for pip install -e. Main problem was that tests directory was missing Have a look at the main branch and let me know if you have any suggestions. This PR is no longer needed. |
@selimsami Excellent, it seems to me that the changes are fine. Let's see if the build will succeed. |
No description provided.