mindersec / minder

Software Supply Chain Security Platform

https://minder-docs.stacklok.dev/

Apache License 2.0

295 stars 43 forks source link

Implement `BuildDataSourceRegistry` in data sources service #5050

Closed JAORMX closed 3 days ago

JAORMX commented 4 days ago

Summary

This implements the aforementioned missing function.

Fixes https://github.com/mindersec/minder/issues/5048

Change Type

Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing

Outline how the changes were tested, including steps to reproduce and any relevant configurations. Attach screenshots if helpful.

Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[x] Checked that related changes are merged.

coveralls commented 3 days ago

coverage: 54.605% (+0.2%) from 54.42% when pulling a3cb677f7caee66a4812d37508854673dc68c10c on JAORMX:BuildDataSourceRegistry-ds into 43d02cd59e6753eb3d589a5a64c83c3f7ec12154 on mindersec:main.