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
Eduu code reorganized for package format #264
Conversation
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.
but why?
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.
the code not workimg
But why no? |
@AndrielFR but why yes? |
Overall it looks good, but for eduu.database I would move it into a database.py file, since there is just one file. |
I left it in a folder because it creates the |
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
since it is more meaningful
This pull request introduces 2 alerts when merging f0f75df into 6237387 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging c65c5be into 6237387 - view on LGTM.com new alerts:
|
…username` _this will fix the `PYL-W0212` issue from DeepSource_
_Based on Codacy and DeepSource analysis_
I did this reorganization using Black and Isort and at Alisson's request, I did not test exactly everything but it seems to me to be all working
, all GitHub workflows will fail because the location of the.version.txt
has been changed then Alisson should change the workflow configuration file