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.

Sign up for GitHub

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

Jump to bottom

refactor: cvedb.py schema checks #4242

Open
terriko opened this issue Jul 3, 2024 · 0 comments · May be fixed by #4568
Open

refactor: cvedb.py schema checks #4242

terriko opened this issue Jul 3, 2024 · 0 comments · May be fixed by #4568
Labels
hackathon Issues for folk participating in the Open Ecosystems hackathon

Comments

@terriko
Copy link
Contributor

terriko commented Jul 3, 2024

Our current schema check requires both the table name and a schema as arguments, but we could switch this to make use of the fact that all the schemas are stored in self.TABLE_SCHEMAS[table_name]

You can see some similar refactoring I did in #4241 for the initialization code.
@terriko terriko added the hackathon Issues for folk participating in the Open Ecosystems hackathon label Jul 3, 2024
This was referenced Nov 12, 2024
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hackathon Issues for folk participating in the Open Ecosystems hackathon
Projects
None yet
Milestone
No milestone
1 participant
@terriko