-
Notifications
You must be signed in to change notification settings - Fork 160
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
Deleting all legacy PACKAGE.py #2247
Conversation
Some tests are failing, reporting a failure of |
The |
Removed PackageSetup class
Done as suggested, kindly check the PR |
Removed GangaCore PACKAGE.py dependency
Removed GangaCore PACKAGE.py dependency
Removed GangaCore PACKAGE.py dependency
We see a set of failures in the testing here, that I assume is a side effect of the code that has been removed.
I think the first step is to understand the warinings and the reason for the failure of the import. The nthe fix might actually be easy. |
I think this has gone in the wrong direction - we want to keep the Notebook, not remove it. |
I'm keeping it, just omitting the line that is giving error on import, I'll fix it asap! |
I'm not sure I understand - it gives you the error because you deleted the file. You need to keep the file and that import. |
@souravpy I fixed the collision caused by the file that was removed/restored. When developing code on a fork, please use a branch rather than the just the |
Deleted following files as remarked