Skip to content

IEBH-353: Sync with the latest changes#9

Merged
vmoshynskyi merged 1 commit intomainfrom
IEBH-353
Nov 24, 2025
Merged

IEBH-353: Sync with the latest changes#9
vmoshynskyi merged 1 commit 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 the service with the latest changes from the pilot repository, updating dependencies, modernizing code syntax, and removing deprecated Vault configuration.

  • Updates Python version constraint from ^3.9 to >=3.10,<3.11 and service version from 2.2.6 to 2.2.9
  • Removes Vault configuration code and related dependencies
  • Modernizes type hints to use Python 3.10+ union syntax (X | None instead of Optional[X])

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Updates service version to 2.2.9 and restricts Python to 3.10.x
app/config.py Removes Vault configuration functions and updates version string
app/commons/notification_service/models.py Modernizes type hints to PEP 604 union syntax and converts string formatting to f-string
app/commons/notification_service/client.py Updates type hints to use modern union syntax
app/main.py Capitalizes first letter of docstring
Dockerfile Updates base image from python-3.10.12-v2 to python-3.10.14-v1
.pre-commit-config.yaml Updates docformatter from v1.7.5 to v1.7.7
.github/workflows/hdc-pipeline.yml Adds new HDC CI/CD pipeline workflow with test, build, and deploy jobs
.env.example Removes Vault-related environment variable examples
tests/init.py, app/routers/init.py, app/routers/v1/init.py, app/resources/init.py, app/models/init.py, app/init.py, app/commons/pipeline_ops/init.py, migrations/versions/396048c16b59_adding_copy_request_tables.py Removes trailing blank lines for code style consistency
app/resources/error_handler.py Adjusts blank line placement for PEP 8 compliance

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

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py00100% 
api_registry.py00100% 
config.py00100% 
logger.py00100% 
main.py0194%41
commons/meta_services
   __init__.py0873%17–18, 20–21, 30–31, 46–47
   models.py00100% 
commons/notification_service
   client.py0295%84–85
   models.py01186%62, 65, 69, 73, 85, 89, 93, 97–98, 113, 117
commons/notifier_service
   email_service.py0287%15, 24
commons/pipeline_ops
   __init__.py00100% 
   copy.py0378%42–44
commons/project_services
   __init__.py00100% 
commons/psql_services
   __init__.py02167%15–16, 18–22, 24–25, 37, 59–60, 62–66, 68–69, 79, 81
models
   __init__.py00100% 
   base.py00100% 
   copy_request.py0792%31, 63–64, 83, 96, 110, 148
   copy_request_sql.py00100% 
resources
   __init__.py00100% 
   error_handler.py0250%10–11
routers
   __init__.py00100% 
   api_root.py0183%17
routers/v1
   __init__.py00100% 
routers/v1/api_copy_request
   api_copy_request.py02889%75, 79, 128, 133, 171, 178, 190–191, 231, 241–243, 246–248, 252, 263, 349, 423–427, 429–433
   request_notify.py0196%22
routers/v1/api_health
   api_health.py01945%22–27, 29, 33–39, 41, 51–54
TOTAL84710687% 

@vmoshynskyi vmoshynskyi merged commit 116811f into main Nov 24, 2025
13 checks passed
@vmoshynskyi vmoshynskyi deleted the IEBH-353 branch November 24, 2025 17:12
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