Skip to content

Conversation

@heliocastro
Copy link
Owner

No description provided.

Signed-off-by: Helio Chissini de Castro <helio.chissini.de.castro@cariad.technology>
@heliocastro heliocastro self-assigned this Nov 10, 2025
Copilot AI review requested due to automatic review settings November 10, 2025 14:04
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR bumps the version to 0.4.1 and refactors the HashAlgorithm class from a simple Enum to a Pydantic BaseModel, enabling more flexible validation and string-based initialization. It also fixes a typo in test data and standardizes quote usage.

  • Refactored HashAlgorithm from Enum to Pydantic BaseModel with aliases, empty values, and validation support
  • Added class methods from_string() and create() for flexible algorithm detection
  • Fixed typo "Giot" → "Git" in test data and updated hash examples

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
uv.lock Updated version from 0.4.0 to 0.4.1
pyproject.toml Updated version from 0.4.0 to 0.4.1
src/ort/models/hash_algorithm.py Refactored from Enum to Pydantic BaseModel with validation and class methods
tests/data/example_simple_curation.yml Standardized quotes and updated hash examples; inconsistent algorithm naming
tests/data/example_curations.yml Fixed typo 'Giot' to 'Git'

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@heliocastro heliocastro merged commit 500dc6c into heliocastro:main Nov 10, 2025
19 checks passed
@heliocastro heliocastro deleted the fix/hash_algo branch November 10, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant