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

DATABASE: refactor rampdb.tools #107

Closed
glemaitre opened this issue Dec 23, 2018 · 3 comments · Fixed by #117
Closed

DATABASE: refactor rampdb.tools #107

glemaitre opened this issue Dec 23, 2018 · 3 comments · Fixed by #117

Comments

@glemaitre
Copy link
Collaborator

glemaitre commented Dec 23, 2018

The current code organisation in rampdb.tools is suboptimal. While it is interested to have query to simplify the code, I think that we should split the api into specific packages. The tools module is also a bit mysterious for the moment.

@kegl
Copy link
Collaborator

kegl commented Jan 9, 2019

Can you be more specific pls. I simply collected all scripts querying and writing into the DB into a file. What would be a better way to organize this?

@glemaitre
Copy link
Collaborator Author

You can check the PR.

Basically organize the tools to deal with submission, event, team, user, etc.
So I organized them in different modules.

@kegl
Copy link
Collaborator

kegl commented Jan 9, 2019

OK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants