Skip to content

Conversation

willmurphyscode
Copy link
Contributor

Add a method for validating when models or blogs change in the v6 schema. Generate two schema files, a JSON schema describing the validation of blobs in the DB, and a SQL schema describing the GORM models.

Add a method for validating when models or blogs change in the v6
schema. Generate two schema files, a JSON schema describing the
validation of blobs in the DB, and a SQL schema describing the GORM
models.

Signed-off-by: Will Murphy <[email protected]>
@willmurphyscode willmurphyscode self-assigned this Sep 29, 2025
@willmurphyscode willmurphyscode moved this to In Progress in OSS Sep 29, 2025
Signed-off-by: Will Murphy <[email protected]>
@willmurphyscode willmurphyscode marked this pull request as ready for review September 30, 2025 14:30
@willmurphyscode willmurphyscode moved this from In Progress to In Review in OSS Sep 30, 2025
@spiffcs
Copy link
Contributor

spiffcs commented Sep 30, 2025

We might want to add a README.md or some other docs at schema/grype/db/ explaining:
- What these files are for
- How to regenerate them
- What to do when schema changes
- Why both versioned and -latest files exist (highlight the drift comparison in ci?)

Copy link
Contributor

@wagoodman wagoodman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 !!

I second what chris is saying around either a readme or more docs in an existing place.

Signed-off-by: Will Murphy <[email protected]>
@willmurphyscode willmurphyscode merged commit 1c1070d into main Oct 1, 2025
12 checks passed
@willmurphyscode willmurphyscode deleted the chore-db-schema branch October 1, 2025 13:25
@github-project-automation github-project-automation bot moved this from In Review to Done in OSS Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants