Skip to content

IEBH-353: Sync with the latest changes#10

Merged
vmoshynskyi merged 2 commits intomainfrom
IEBH-353
Nov 21, 2025
Merged

IEBH-353: Sync with the latest changes#10
vmoshynskyi merged 2 commits intomainfrom
IEBH-353

Conversation

@vmoshynskyi
Copy link
Member

Add the latest changes from the pilot repo.

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 synchronizes changes from the pilot repository, modernizing the codebase to use Python 3.10+ features and updating dependencies. The changes include type hint modernization using PEP 604 syntax, removal of vault configuration, and dependency updates.

Key Changes:

  • Upgraded Python requirement from 3.9+ to 3.10+ only
  • Modernized type hints to use PEP 604 union syntax (X | Y instead of Union[X, Y] and X | None instead of Optional[X])
  • Updated critical dependencies (asyncpg, pillow, pydantic)

Reviewed changes

Copilot reviewed 39 out of 40 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
pyproject.toml Updated Python version to 3.10+ and upgraded dependencies (asyncpg, pillow, pydantic)
project/config.py Removed VaultConfig class and vault-related configuration
project/components/*.py Modernized type hints to PEP 604 syntax and replaced Type[X] with type[X]
project/services/auth.py Updated type hints to use PEP 604 syntax
tests/**/*.py Updated type hints and refactored tests to use inspect.get_annotations()
migrations/versions/*.py Removed extra blank lines and fixed docstring capitalization
.pre-commit-config.yaml Enabled docformatter hook (v1.7.7)
.github/workflows/hdc-pipeline.yml Added new CI/CD pipeline workflow
.env.example Removed vault-related environment variables

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

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py00100% 
__main__.py050%7, 9, 11–13
app.py0690%111, 124–125, 127, 129, 133
config.py00100% 
logger.py00100% 
components
   __init__.py00100% 
   crud.py0694%90, 117, 125, 149–150, 152
   db_model.py00100% 
   exceptions.py0692%29, 36, 43, 107, 111, 115
   filtering.py0191%30
   models.py00100% 
   pagination.py00100% 
   parameters.py0196%63
   schemas.py0196%30
   sorting.py00100% 
   types.py00100% 
components/health
   __init__.py00100% 
   db_checker.py00100% 
   dependencies.py00100% 
   views.py00100% 
components/object_storage
   __init__.py00100% 
   policy.py0688%78–80, 82–84
   policy_templates.py00100% 
   s3.py0197%49
components/project
   __init__.py00100% 
   crud.py00100% 
   dependencies.py00100% 
   filtering.py00100% 
   logo_uploader.py0981%34–36, 55–57, 64–66
   models.py00100% 
   object_storage_manager.py0391%27, 30–31
   parameters.py0197%46
   schemas.py0394%77–78, 82
   views.py00100% 
components/resource_request
   __init__.py00100% 
   crud.py00100% 
   dependencies.py00100% 
   filtering.py00100% 
   models.py00100% 
   parameters.py00100% 
   schemas.py00100% 
   sorting.py00100% 
   views.py00100% 
components/workbench
   __init__.py00100% 
   crud.py00100% 
   dependencies.py00100% 
   filtering.py00100% 
   models.py00100% 
   parameters.py00100% 
   schemas.py00100% 
   views.py00100% 
dependencies
   __init__.py00100% 
   db.py0575%36, 38–40, 42
   s3.py00100% 
services
   __init__.py00100% 
   auth.py01284%46–48, 69–71, 90–92, 117–119
   metadata.py0491%34, 82–84
TOTAL13807094% 

@vmoshynskyi vmoshynskyi merged commit c0ca53a into main Nov 21, 2025
7 checks passed
@vmoshynskyi vmoshynskyi deleted the IEBH-353 branch November 21, 2025 22:06
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