Skip to content

IEBH-353: Sync with the latest changes#9

Merged
vmoshynskyi merged 3 commits intomainfrom
IEBH-353
Nov 24, 2025
Merged

IEBH-353: Sync with the latest changes#9
vmoshynskyi merged 3 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 Python 3.10+ standards and updating dependencies. The changes include adopting modern type hint syntax (PEP 604), updating the Python version constraint, upgrading pydantic, removing deprecated Vault configuration, and adding a new GitHub Actions CI/CD pipeline.

  • Adopts Python 3.10+ type hint syntax using PEP 604 union operators and built-in generic types
  • Updates Python version constraint to >=3.10,<3.11 and upgrades pydantic from 1.8.2 to 1.10.19
  • Removes deprecated Vault configuration infrastructure and replaces Jenkinsfile with GitHub Actions workflow

Reviewed changes

Copilot reviewed 24 out of 26 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pyproject.toml Updates project version to 2.5.8, Python constraint to >=3.10,<3.11, pydantic to 1.10.19, removes explicit typing_extensions dependency
poetry.lock Updates lock file with new pydantic 1.10.19 and typing-extensions 4.12.2 versions, but contains metadata inconsistency with Python version
dataops/config.py Removes VaultConfig class and vault-related configuration, updates VERSION to 2.5.7
tests/fixtures/components/resource_operations.py Imports Callable from collections.abc instead of typing
migrations/env.py Removes typing.Optional import, uses PEP 604 union syntax (str | None)
migrations/versions/*.py Removes blank lines, capitalizes docstring to follow proper sentence structure
dataops/components/task_stream/schemas.py Replaces typing imports with PEP 604 union syntax and built-in generic types
dataops/components/task_stream/parsing.py Replaces typing.Union with PEP 604 union syntax
dataops/components/task_stream/views.py Removes unused logger import
dataops/components/task_dispatch/crud.py Replaces typing imports with modern syntax, converts .format() to f-strings
dataops/components/resource_operations/schemas.py Replaces typing imports with built-in generic types and PEP 604 syntax
dataops/components/resource_operations/filtering.py Replaces typing imports with built-in generic types
dataops/components/resource_operations/crud/base.py Replaces typing.List with built-in list type
dataops/components/resource_lock/schemas.py Replaces typing imports with built-in generic types and PEP 604 syntax
dataops/components/resource_lock/cache.py Replaces typing.List with built-in list type
dataops/components/resource_lock/views.py Removes unused logger import
dataops/components/exceptions.py Imports Sequence from collections.abc instead of typing
dataops/components/crud.py Replaces typing imports with modern syntax, converts .format() to f-strings
dataops/components/cache.py Replaces typing imports with PEP 604 union syntax
dataops/components/archive_preview/views.py Replaces typing.Union with PEP 604 union syntax
Jenkinsfile Deletes entire Jenkinsfile (124 lines) as CI/CD is moving to GitHub Actions
.pre-commit-config.yaml Uncomments docformatter hook and updates version to v1.7.7
.gitignore Adds .python-version to ignore list for pyenv support
.github/workflows/hdc-pipeline.yml Adds new GitHub Actions CI/CD pipeline with test, build, and deploy jobs
.env.example Removes 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.py01284%88, 120, 124, 129–130, 132–134, 136–137, 139, 143
config.py00100% 
logger.py00100% 
components
   __init__.py00100% 
   authorization.py0190%21
   cache.py00100% 
   crud.py02380%87, 106, 142, 146, 150–152, 156–158, 162, 166, 170–176, 180–181, 185, 189
   db_model.py00100% 
   exceptions.py01982%31, 38, 45, 61, 65, 69, 125, 129, 133, 156, 160, 164, 168–170, 178, 186, 190, 194
   schemas.py00100% 
components/archive_preview
   __init__.py00100% 
   crud.py00100% 
   dependencies.py00100% 
   models.py00100% 
   schemas.py00100% 
   views.py00100% 
components/health
   __init__.py00100% 
   dependencies.py00100% 
   resource_checkers.py0676%27–28, 30, 45–46, 48
   views.py00100% 
components/resource_lock
   __init__.py00100% 
   cache.py0592%36–37, 43, 91, 155
   dependencies.py00100% 
   schemas.py00100% 
   views.py00100% 
components/resource_operations
   __init__.py00100% 
   dependencies.py00100% 
   filtering.py00100% 
   schemas.py00100% 
   views.py00100% 
components/resource_operations/crud
   __init__.py00100% 
   base.py00100% 
   dispatcher.py00100% 
components/task_dispatch
   __init__.py00100% 
   crud.py0295%96–97
   dependencies.py00100% 
   schemas.py00100% 
   sorting.py00100% 
   views.py00100% 
components/task_stream
   __init__.py00100% 
   crud.py0790%40–42, 64–66, 95
   dependencies.py00100% 
   parsing.py0388%38–40
   schemas.py00100% 
   views.py00100% 
dependencies
   __init__.py00100% 
   auth.py0286%23–24
   db.py0573%37–40, 42
   redis.py00100% 
services
   __init__.py00100% 
   metadata.py0585%41–42, 48–50
   queue.py0678%36–38, 40–42
TOTAL129210192% 

@vmoshynskyi vmoshynskyi merged commit 84f2b6e into main Nov 24, 2025
7 checks passed
@vmoshynskyi vmoshynskyi deleted the IEBH-353 branch November 24, 2025 16:37
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