Add analyzer_versions to AppMetadata#268
Add analyzer_versions to AppMetadata#268google-labs-jules[bot] wants to merge 1 commit intodevelopfrom
Conversation
This change adds `analyzer_versions` (plural) to `AppMetadata` to support multi-model or multi-family apps. It is an optional map of analyzer IDs to their versions. This aligns with the latest proposal in issue #251, discarding the previous `hf-` prefix validation requirement for `analyzer_version`.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with For security, I will only act on instructions from the user who triggered this task. New to Jules? Learn more at jules.google/docs. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #268 +/- ##
===========================================
+ Coverage 61.33% 61.37% +0.04%
===========================================
Files 6 6
Lines 812 813 +1
===========================================
+ Hits 498 499 +1
Misses 314 314
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Implemented the final proposal for issue #251 by adding the
analyzer_versionsfield toAppMetadatainclams/appmetadata/__init__.py. This new field allows multi-model apps to specify versions for multiple components. The singularanalyzer_versionfield remains for single-model apps, and no validation is enforced on it (specifically, thehf-format proposal was discarded). Added comprehensive tests intests/test_appmetadata.pyto verify the new field and ensure existing functionality remains intact.PR created automatically by Jules for task 17929783131841457298 started by @keighrim